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

          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Priority P2
          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.
          yip.ng made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 1.8.2-EE-GA_P03 [ 10251 ]
          Resolution Fixed [ 1 ]
          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
          yip.ng made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          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.
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #24262 Mon Mar 28 10:02:36 MDT 2011 yip.ng ICE-6704: Data Paginator vertical alignment broken.
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/datapaginator/DataPaginatorRenderer.java
          yip.ng made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          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
          yip.ng made changes -
          Attachment screenshot-03.png [ 13021 ]
          Ken Fyten made changes -
          Salesforce Case []
          Assignee Priority P2
          Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #24258 Fri Mar 25 17:27:55 MDT 2011 yip.ng ICE-6704: Data Paginator vertical alignment broken.
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/datapaginator/DataPaginatorRenderer.java
          Carlo Guglielmin made changes -
          Link This issue duplicates ICE-6495 [ ICE-6495 ]
          Carlo Guglielmin made changes -
          Link This issue duplicates ICE-6495 [ ICE-6495 ]
          Carlo Guglielmin made changes -
          Link This issue duplicates ICE-5959 [ ICE-5959 ]
          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.
          Carlo Guglielmin made changes -
          Assignee Ken Fyten [ ken.fyten ]
          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.
          Carlo Guglielmin made changes -
          Field Original Value New Value
          Attachment 6704-firebug.png [ 13018 ]
          Attachment 6704-vertical.png [ 13019 ]
          Hide
          Carlo Guglielmin added a comment -

          Added screenshots.

          Show
          Carlo Guglielmin added a comment - Added screenshots.
          Carlo Guglielmin created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved: