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

              duplicates

              Bug - A problem which impairs or prevents the functions of the product. ICE-5959 Datapaginator: Wrong rendering with vertical setting

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Carlo Guglielmin created issue -
                Hide
                Permalink
                Carlo Guglielmin added a comment -

                Added screenshots.

                Show
                Carlo Guglielmin added a comment - Added screenshots.
                Carlo Guglielmin made changes -
                Field Original Value New Value
                Attachment 6704-firebug.png [ 13018 ]
                Attachment 6704-vertical.png [ 13019 ]
                Hide
                Permalink
                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 -
                Assignee Ken Fyten [ ken.fyten ]
                Hide
                Permalink
                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 -
                Link This issue duplicates ICE-5959 [ ICE-5959 ]
                Carlo Guglielmin made changes -
                Link This issue duplicates ICE-6495 [ ICE-6495 ]
                Carlo Guglielmin made changes -
                Link This issue duplicates ICE-6495 [ ICE-6495 ]
                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
                Ken Fyten made changes -
                Salesforce Case []
                Assignee Priority P2
                Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
                yip.ng made changes -
                Attachment screenshot-03.png [ 13021 ]
                Hide
                Permalink
                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 -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                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
                Hide
                Permalink
                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.
                yip.ng made changes -
                Resolution Fixed [ 1 ]
                Status Resolved [ 5 ] Reopened [ 4 ]
                Hide
                Permalink
                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 -
                Status Reopened [ 4 ] Resolved [ 5 ]
                Fix Version/s 1.8.2-EE-GA_P03 [ 10251 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                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.
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]
                Assignee Priority P2

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: