ICEfaces
  1. ICEfaces
  2. ICE-9317

ace:dataTable - ICE-8237 - Chrome Unaligned

          Details

          • Assignee Priority:
            P1

            Description

            The ace:dataTable regression for ICE-8237 fails:

            In Chrome: the columns are slightly misaligned when rendering the table - see attached screen shot.

              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 [ 10572 ]
              Fix Version/s 3.4 [ 10770 ]
              Carmen Cristurean made changes -
              Description The ace:dataTable regression for ICE-8237 fails:

              In Chrome: the columns are slightly misaligned (not sure how much of an issue this is) when rendering the table - see attached screen shot.

              All browsers: a styling issue is visible when sorting items - half of the column header that is clicked for sorting changes color, see attached screen shot.

              In IE8: there are two table headers and a JS error:
              Message: Object doesn't support this property or method
              Line: 434
              Char: 8455
              Code: 0
              URI: http://localhost:8080/dataTable/javax.faces.resource/util/ace-jquery.js.jsf?ln=icefaces.ace&v=3_4_0_130603
              		The ace:dataTable regression for ICE-8237 fails:

              In Chrome: the columns are slightly misaligned when rendering the table - see attached screen shot.

              All browsers: a styling issue is visible when sorting items - half of the column header that is clicked for sorting changes color, see attached screen shot.

              In IE8: there are two table headers and a JS error:
              Message: Object doesn't support this property or method
              Line: 434
              Char: 8455
              Code: 0
              URI: http://localhost:8080/dataTable/javax.faces.resource/util/ace-jquery.js.jsf?ln=icefaces.ace&v=3_4_0_130603
              Carmen Cristurean made changes -
              Attachment ICE-8237-chrome.png [ 16013 ]
              Ken Fyten made changes -
              Assignee Nils Lundquist [ nils.lundquist ]
              Assignee Priority P1 [ 10010 ]
              Nils Lundquist made changes -
              Summary Regression - ace:dataTable - ICE-8237 test failure ace:dataTable - ICE-8237 - Chrome Unaligned
              Nils Lundquist made changes -
              Description The ace:dataTable regression for ICE-8237 fails:

              In Chrome: the columns are slightly misaligned when rendering the table - see attached screen shot.

              All browsers: a styling issue is visible when sorting items - half of the column header that is clicked for sorting changes color, see attached screen shot.

              In IE8: there are two table headers and a JS error:
              Message: Object doesn't support this property or method
              Line: 434
              Char: 8455
              Code: 0
              URI: http://localhost:8080/dataTable/javax.faces.resource/util/ace-jquery.js.jsf?ln=icefaces.ace&v=3_4_0_130603
              		The ace:dataTable regression for ICE-8237 fails:

              In Chrome: the columns are slightly misaligned when rendering the table - see attached screen shot.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #36020 Thu Jun 06 16:35:35 MDT 2013 nils.lundquist ICE-9317 - Altered regex to match multicharacter indexes.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #36020
              Committed by nils.lundquist
              A minute ago
              ICE-9317 - Altered regex to match multicharacter indexes.

              Show
              Nils Lundquist added a comment - Revision #36020 Committed by nils.lundquist A minute ago ICE-9317 - Altered regex to match multicharacter indexes.
              Nils Lundquist made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              QA reports the following regression that seems possibly related to this commit:

              Tests have been switched off the ICEfaces EE 3.3.0.GA specific branch (repo/icefaces-ee3/tags/icefaces-ee-3.3.0.GA/icefaces-ee).

              ICEfaces-EE 3.3.0.GA/icefaces-ee revision# 34706
              Server: tomcat6
              Browser: FF3.6, IE8, GoogleChrome27

              ace:dataTable (Chrome, IE) (In Chrome ICE-8237 test fails due to misalignment. Columns are misaligned when first rendering the table. After clicking to sort on column headers, the alignment is improved.)

              Show
              Ken Fyten added a comment - QA reports the following regression that seems possibly related to this commit: Tests have been switched off the ICEfaces EE 3.3.0.GA specific branch (repo/icefaces-ee3/tags/icefaces-ee-3.3.0.GA/icefaces-ee). ICEfaces-EE 3.3.0.GA/icefaces-ee revision# 34706 Server: tomcat6 Browser: FF3.6, IE8, GoogleChrome27 ace:dataTable (Chrome, IE) (In Chrome ICE-8237 test fails due to misalignment. Columns are misaligned when first rendering the table. After clicking to sort on column headers, the alignment is improved.)
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #36092 Mon Jun 10 11:19:24 MDT 2013 nils.lundquist ICE-9317- Fixed regex typo
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #36092
              Committed by nils.lundquist
              5 minutes ago
              ICE-9317- Fixed regex typo

              Show
              Nils Lundquist added a comment - Revision #36092 Committed by nils.lundquist 5 minutes ago ICE-9317 - Fixed regex typo
              Nils Lundquist made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Nils Lundquist added a comment -

              Regex ui-col-([0-9])+ should've been ui-col-([0-9]+)

              Show
              Nils Lundquist added a comment - Regex ui-col-( [0-9] )+ should've been ui-col-( [0-9] +)
              Hide
              Permalink
              Cruz Miraback added a comment -

              Issue can be reproduced in Chrome27, firefox21, and IE10 using the ICEfaces-EE-3.3.0 Build8 bundle. See screenshot, columns are misaligned in this way in all browsers.

              Show
              Cruz Miraback added a comment - Issue can be reproduced in Chrome27, firefox21, and IE10 using the ICEfaces-EE-3.3.0 Build8 bundle. See screenshot, columns are misaligned in this way in all browsers.
              Cruz Miraback made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Cruz Miraback made changes -
              Attachment screenshot-1.jpg [ 16035 ]
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #36165
              Committed by nils.lundquist
              Moments ago
              ICE-9317 - Added 100% width to tables when scrollbars conditionally added and avoided

              Show
              Nils Lundquist added a comment - Revision #36165 Committed by nils.lundquist Moments ago ICE-9317 - Added 100% width to tables when scrollbars conditionally added and avoided
              Nils Lundquist made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #36165 Tue Jun 11 17:01:09 MDT 2013 nils.lundquist ICE-9317 - Added 100% width to tables when scrollbars conditionally added and avoided
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Cruz Miraback added a comment -

              Confirmed fixed in Chrome27, Firefox21, and IE10 using ICEfaces-EE-3.3.0 Build10.

              Show
              Cruz Miraback added a comment - Confirmed fixed in Chrome27, Firefox21, and IE10 using ICEfaces-EE-3.3.0 Build10.
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11382 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Carmen Cristurean
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: