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

              is duplicated by

              Improvement - An improvement or enhancement to an existing feature or task. ICE-6689 CLONE -selectOneRadio renders a table with default cell padding and spacing

              • Minor - Minor loss of function, or other problem where easy workaround is present.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Ken Fyten added a comment -

                In order to maintain complete backwards compatiblitiy with the existing rendering and component spacing we could add the padding back into our default theme style sheets so that we don't mess up any existing applications.

                The new rule would likely look like:

                .iceSelOneRb table tr td

                { padding: 1px; margin: 1px; }
                Show
                Ken Fyten added a comment - In order to maintain complete backwards compatiblitiy with the existing rendering and component spacing we could add the padding back into our default theme style sheets so that we don't mess up any existing applications. The new rule would likely look like: .iceSelOneRb table tr td { padding: 1px; margin: 1px; }
                Hide
                Permalink
                Patrick Corless added a comment -

                I suspect the margin: 1px maybe causing this. Can you confirm?

                Show
                Patrick Corless added a comment - I suspect the margin: 1px maybe causing this. Can you confirm?
                Hide
                Permalink
                yip.ng added a comment -

                The margin property doesn't seem to have any effect on the layout. I tried changing it to a large value, but the layout remained the same. Also, the default value was "0" in Firefox and "auto" in IE. So, if it is for backwards compatibility, we should probably just leave it out.

                Show
                yip.ng added a comment - The margin property doesn't seem to have any effect on the layout. I tried changing it to a large value, but the layout remained the same. Also, the default value was "0" in Firefox and "auto" in IE. So, if it is for backwards compatibility, we should probably just leave it out.

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: