ICEfaces
  1. ICEfaces
  2. ICE-3140

selectOneRadio renders a table with default cell padding and spacing

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7
    • Fix Version/s: 1.7.1
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      any uses of selectOneRadio

      Description

      The selectOneRadio renderer uses a HTML table element to layout its child components. The table element does not specify either cellpadding or cellspacing attributes which make styling the component difficult. Both cellpadding and cellspacing default to 1px which means that the selectOneRadio component will be offset as much as 4px in the vertical with any label on the same plain.

      I recommend that the cellpading and cellspacing attributes be set with initial values of zero. CSS can then be used by the end user to specify the equivalent L&F, no changes will be needed in our default themes as the end result of the renderer is favourable in our demo application.

        Issue Links

          Activity

          Patrick Corless created issue -
          Patrick Corless made changes -
          Field Original Value New Value
          Fix Version/s 1.7.1 [ 10122 ]
          Assignee Ken Fyten [ ken.fyten ]
          Priority Major [ 3 ] Minor [ 4 ]
          Ken Fyten made changes -
          Assignee Priority P3
          Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
          made changes -
          Attachment screenshot-1.jpg [ 11044 ]
          yip.ng made changes -
          Attachment screenshot-2.jpg [ 11047 ]
          yip.ng made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Priority P3
          Assignee Yip Ng [ yip.ng ]
          Ken Fyten made changes -
          Link This issue is duplicated by ICE-6689 [ ICE-6689 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Patrick Corless
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: