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

        Hide
        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 .
        Hide
        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).
        Hide
        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).
        Hide
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: