ICEfaces
  1. ICEfaces
  2. ICE-11112

ace:dataTable - Scrollable table alignment issue when used in an ace:panelExpansion

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.3.0.GA_P03, EE-4.1.0.GA
          • Fix Version/s: 4.2.BETA, 4.2, EE-3.3.0.GA_P05
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            Chrome (Customer reports issue in Chrome, Firefox, and IE11)

            Description

            If a scrollable ace:dataTable is placed in an ace:panelExpansion and the table that has the expansion is also a scrollable ace:dataTable, the columns of the table in the expansion are not aligned correctly.

            In my testing I can only see this issue in Chrome but the customer is reporting this in Chrome, Firefox, and IE11.

            If the scrollable values are removed from the parent table the inner table aligns correctly.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              Arran Mccullough added a comment -

              Another issue is that the contents of the first row of the inner table are shifted to the left compared to the other rows contents.

              Show
              Arran Mccullough added a comment - Another issue is that the contents of the first row of the inner table are shifted to the left compared to the other rows contents.
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case to reproduce the issue.

              Steps:

              • Load welcomeICEfaces.jsf
              • Expand one of the rows. The inner table is displayed with the columns not aligned.
              Show
              Arran Mccullough added a comment - Attached test case to reproduce the issue. Steps: Load welcomeICEfaces.jsf Expand one of the rows. The inner table is displayed with the columns not aligned.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case13819Example3.war [ 22259 ]
              Attachment Case13819Example3.zip [ 22260 ]
              Ken Fyten made changes -
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s 4.2 [ 12870 ]
              Fix Version/s EE-3.3.0.GA_P05 [ 13082 ]
              Assignee Priority P1 [ 10010 ]
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed fix to the 4.0 trunk at revision 49133 and to the 3.3 EE maintenance branch at revision 49134.

              The problem was that the dynamically calculated column widths of the outer table were being applied to the columns of the inner tables. So, the fix consisted of adding more specificity to the dynamically generated CSS rules to make sure that these dynamically calculated column widths aren't applied to inner tables' columns.

              Show
              Arturo Zambrano added a comment - Committed fix to the 4.0 trunk at revision 49133 and to the 3.3 EE maintenance branch at revision 49134. The problem was that the dynamically calculated column widths of the outer table were being applied to the columns of the inner tables. So, the fix consisted of adding more specificity to the dynamically generated CSS rules to make sure that these dynamically calculated column widths aren't applied to inner tables' columns.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #49133 Tue Aug 30 12:04:53 MDT 2016 art.zambrano ICE-11112 added more specificity to the dynamically generated CSS rules to make sure that these dynamically calculated column widths aren't applied to inner tables' columns
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Liana Munroe added a comment -

              Verified ICEfaces 4 trunk, ee-3.3.0 maintenance branch r49141. Tomcat 8, IE edge, 11, 10, 8, 7, FF 47, Chrome 52, using attached test app as well as QA dataTable and showcase regression tests.

              Show
              Liana Munroe added a comment - Verified ICEfaces 4 trunk, ee-3.3.0 maintenance branch r49141. Tomcat 8, IE edge, 11, 10, 8, 7, FF 47, Chrome 52, using attached test app as well as QA dataTable and showcase regression tests.
              Ken Fyten made changes -
              Fix Version/s 4.2.BETA [ 13091 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Arran Mccullough
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: