ICEfaces
  1. ICEfaces
  2. ICE-6704

Data Paginator vertical alignment broken

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: 2.0.1, EE-1.8.2.GA_P03
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      Tomcat 6, Firefox 3.x, ICEfaces 2.0 or ICEfaces 2 trunk rev 24256.

      Description

      The dataPaginator component is displaying incorrectly when vertical="true" is set.

      The next, fastforward, and fastend buttons are displayed in the same table row as the pages (see attached screenshot). The rendered code (see attached Firebug screenshot) clearly shows that an additional <tr> is needed, as the pages and last three buttons are all in the same table row.
      1. 6704-firebug.png
        42 kB
      2. 6704-vertical.png
        34 kB
      3. screenshot-03.png
        192 kB

        Issue Links

          Activity

          Hide
          Carlo Guglielmin added a comment -

          Added screenshots.

          Show
          Carlo Guglielmin added a comment - Added screenshots.
          Hide
          Carlo Guglielmin added a comment -

          Assigning to Ken so he can delegate to component team as needed.

          Show
          Carlo Guglielmin added a comment - Assigning to Ken so he can delegate to component team as needed.
          Hide
          Carlo Guglielmin added a comment - - edited

          Linked to an old 1.8 bug with a possible fix.

          Show
          Carlo Guglielmin added a comment - - edited Linked to an old 1.8 bug with a possible fix.
          Hide
          yip.ng added a comment -

          Fixed as per suggestion in ICE-5959.

          Revision: 24258


          Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/datapaginator/DataPaginatorRenderer.java

          Show
          yip.ng added a comment - Fixed as per suggestion in ICE-5959 . Revision: 24258 Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/datapaginator/DataPaginatorRenderer.java
          Hide
          yip.ng added a comment -

          Need to fix 1.8 as well.

          Show
          yip.ng added a comment - Need to fix 1.8 as well.
          Hide
          yip.ng added a comment -

          Revision: 24262


          Modified : /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/datapaginator/DataPaginatorRenderer.java

          Show
          yip.ng added a comment - Revision: 24262 Modified : /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/datapaginator/DataPaginatorRenderer.java
          Hide
          Mandeep Hayher added a comment -

          Tested successfully on icefaces1.8 trunk revision# 24720 using FF3.6, IE8 & chrome10.

          Show
          Mandeep Hayher added a comment - Tested successfully on icefaces1.8 trunk revision# 24720 using FF3.6, IE8 & chrome10.

            People

            • Assignee:
              yip.ng
              Reporter:
              Carlo Guglielmin
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: