ICEfaces
  1. ICEfaces
  2. ICE-10040

showcase - ace:dataTable Scrolling demo styling issue (Firefox)

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: EE-3.3.0.GA_P01
          • Fix Version/s: EE-3.3.0.GA_P02, 4.0
          • Component/s: ACE-Components, Sample Apps
          • Labels:
            None
          • Environment:
            Jenkins EE 3 trunk Build # 425/ 427
            Server: Tomcat7
            Showcase Firefox29
          • Assignee Priority:
            P3
          • Affects:
            Sample App./Tutorial
          • Workaround Description:
            Not sure if this is a valid issue or what.

            Description

            showcase > Scrolling (Firefox29 only): see attached screen shot, there is a dataTable styling issue, and the scrollbar is not visible, when entering the minimum allowed table height in pixels (50), and scrolling is enabled; there is also similar wrong styling in the table config panel when opened.

              Activity

              Ascending order - Click to sort in descending order
              Carmen Cristurean created issue -
              Carmen Cristurean made changes -
              Field Original Value New Value
              Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Carmen Cristurean made changes -
              Attachment table-scrolling.PNG [ 17199 ]
              Ken Fyten made changes -
              Workaround Description Not sure if this is a valid issue or what.
              Assignee Arturo Zambrano [ artzambrano ]
              Priority Major [ 3 ] Minor [ 4 ]
              Assignee Priority P3 [ 10012 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #41198 Thu May 22 12:04:41 MDT 2014 art.zambrano ICE-10040 changed scrollable tables' styling from overflow:scroll to overflow-y:scroll and overflow-x:auto
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.css
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed fix to 4.0 trunk at revision 41198 and to 3.3 EE maintenance branch at revision 41197. This was caused by ICE-9908, when the scrollable tables' styling was changed from overflow:auto to overflow:scroll. Now, this was changed again to overflow-y:scroll and overflow-x:auto, since only the vertical scrollbar is required to always appear, in order to solve some of the alignment issues fixed in ICE-9908.

              Show
              Arturo Zambrano added a comment - Committed fix to 4.0 trunk at revision 41198 and to 3.3 EE maintenance branch at revision 41197. This was caused by ICE-9908 , when the scrollable tables' styling was changed from overflow:auto to overflow:scroll. Now, this was changed again to overflow-y:scroll and overflow-x:auto, since only the vertical scrollbar is required to always appear, in order to solve some of the alignment issues fixed in ICE-9908 .
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment - - edited

              EE-3.3.0-maintenance barnch rev. 41200 testing:
              The dataTable Scrolling demo looks good now in Firefox29, however the table config panel, if opened, has also a styling issue in Firefox29 and Chrome35; not an issue in IE10 (image attached).

              Show
              Carmen Cristurean added a comment - - edited EE-3.3.0-maintenance barnch rev. 41200 testing: The dataTable Scrolling demo looks good now in Firefox29, however the table config panel, if opened, has also a styling issue in Firefox29 and Chrome35; not an issue in IE10 (image attached).
              Carmen Cristurean made changes -
              Attachment scrolling-panelConfig.PNG [ 17201 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #41215 Tue May 27 10:30:17 MDT 2014 art.zambrano ICE-10040, ICE-10041 increased minimum height requirement for tables; removed 'rows' and 'paginator' attributes from table in the dialog
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/ace/dataTable/dataTableScrolling.xhtml
              Hide
              Permalink
              Arturo Zambrano added a comment -

              The last issue is not really an issue on our end. It happens in FF when the container height is just too short to be able to display the vertical scrollbar, like the minimum value in the demo (50). Simply using 52 or more avoids the problem. This issue is beyond our control and is easily fixed at the app level. In general, in production scenarios, there's no need to have such a short height for a scrollable container. I couldn't reproduce this issue in Chrome, but I suppose it's the same situation.

              The scrolling demo was modified to allow for a minimum height of 60, so to illustrate an appropriate minimum height for a scrollable table (or any other scrollable container).

              Show
              Arturo Zambrano added a comment - The last issue is not really an issue on our end. It happens in FF when the container height is just too short to be able to display the vertical scrollbar, like the minimum value in the demo (50). Simply using 52 or more avoids the problem. This issue is beyond our control and is easily fixed at the app level. In general, in production scenarios, there's no need to have such a short height for a scrollable container. I couldn't reproduce this issue in Chrome, but I suppose it's the same situation. The scrolling demo was modified to allow for a minimum height of 60, so to illustrate an appropriate minimum height for a scrollable table (or any other scrollable container).
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Verified with EE-3.3.0-maintenance branch rev.41218 in FF29, Chrome35.

              Show
              Carmen Cristurean added a comment - Verified with EE-3.3.0-maintenance branch rev.41218 in FF29, Chrome35.
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Ken Fyten made changes -
              Affects Version/s EE-3.3.0.GA_P01 [ 11174 ]
              Affects Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11382 ]
              Ken Fyten made changes -
              Affects Sample App./Tutorial [ 10001 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Carmen Cristurean
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: