ICEfaces
  1. ICEfaces
  2. ICE-8838

ace:dataTable - paginator doesn't extend for the entire table size when displayed in a scrollable panel

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.0.0.GA_P01, 3.2
          • Fix Version/s: EE-3.2.0.GA, 3.3
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            All
          • Assignee Priority:
            P3
          • Salesforce Case Reference:
          • Workaround Exists:
            Yes
          • Workaround Description:
            Use a fixed "px" width for the ace:dataTable

            Description

            When an ace:dataTable is used in scrollable panel the paginator does not extend for the entire table width. The paginator looks to set its width depending on the parent containers width and not what the dataTable is set as.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case Reference 5007000000QWSjnAAH
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case and image that shows the issue.

              Show
              Arran Mccullough added a comment - Attached test case and image that shows the issue.
              Arran Mccullough made changes -
              Attachment Case11853Example.zip [ 15184 ]
              Attachment Case11853ExampleWAR.zip [ 15185 ]
              Attachment paginator-display-issue.jpg [ 15186 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              For some cases, the existing behavior might be considered a "feature", as it eliminates the need to scroll horizontally to use the paginator controls. Perhaps we should add a new attribute to control the pagination width configuration (window vs table)?

              Show
              Ken Fyten added a comment - For some cases, the existing behavior might be considered a "feature", as it eliminates the need to scroll horizontally to use the paginator controls. Perhaps we should add a new attribute to control the pagination width configuration (window vs table)?
              Ken Fyten made changes -
              Fix Version/s EE-3.2.0.GA [ 10332 ]
              Fix Version/s 3.3 [ 10370 ]
              Assignee Nils Lundquist [ nils.lundquist ]
              Assignee Priority P3 [ 10012 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33011 Wed Jan 09 15:43:50 MST 2013 nils.lundquist ICE-8838 - Removed nowrap from paginator and changed table display to inline block.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/paginator/paginator.css
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.css
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33012 Wed Jan 09 16:02:53 MST 2013 nils.lundquist ICE-8838 - Added width:100% to change the default associated with display inline-block.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.css
              Hide
              Permalink
              Nils Lundquist added a comment -

              The position of the "per page" text in this case is due to the nowrap rule of the paginator. This rule causes all text to stay on the same line even if there is no space for it. I would say the best way of resolving this issue is removing that rule, and allowing the paginator contents to wrap when necessary. Otherwise, by fixing the background for the paginator, in cases when the paginator is overflowing the parent container but the parent container doesn't have a scrollbar, the paginator will blatantly render outside the bounds of the parent container, looking terrible.

              Show
              Nils Lundquist added a comment - The position of the "per page" text in this case is due to the nowrap rule of the paginator. This rule causes all text to stay on the same line even if there is no space for it. I would say the best way of resolving this issue is removing that rule, and allowing the paginator contents to wrap when necessary. Otherwise, by fixing the background for the paginator, in cases when the paginator is overflowing the parent container but the parent container doesn't have a scrollbar, the paginator will blatantly render outside the bounds of the parent container, looking terrible.
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #33011
              Committed by nils.lundquist
              A minute ago ICE-8838 - Removed nowrap from paginator and changed table display to inline block.

              Show
              Nils Lundquist added a comment - Revision #33011 Committed by nils.lundquist A minute ago ICE-8838 - Removed nowrap from paginator and changed table display to inline block.
              Nils Lundquist made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33017 Thu Jan 10 09:59:20 MST 2013 nils.lundquist ICE-8838 - Paginator to block after changing table to 100% width inline-block.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/paginator/paginator.css
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: