Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Invalid
-
Affects Version/s: 3.2
-
Fix Version/s: 3.2
-
Component/s: ACE-Components
-
Labels:None
-
Environment:Icefaces3.0 trunk revision# 31266
Description
Steps to reproduce styling issue
1) Load test application
2) Navigate to one of the following pages:
/views/scrollingGroupHeader.jsf
/views/scrollExpansionTest.jsf
/views/liveScrollingTest.jsf
/views/dataTableDialog.jsf
/views/
/views/
3) Notice the column headers are misaligned, see attached image.
Steps to reproduce JS error:
1) Load test application
2) Navigate to '/views/delayedRowSelectionTest.jsf' or '/views/delayedMultiSelectionTest.jsf'
3) Click back in the browser
4) JS error is thrown:
Message: Object doesn't support this property or method
Line: 1
Char: 36591
Code: 0
URI: http://localhost:8080/dataTable/javax.faces.resource/bridge.js.jsf?v=13636171
or instead
2) Navigate to '/views/
3) Select a filter option from the list
4) Select another
Similar JS error is thrown
Activity
The revision number is uncertain as this test continuously fails in automation (IE) due to selenium issues. I found it by testing manually today when checking for any issues that might be hiding.
Show
Cruz Miraback
added a comment - - edited The revision number is uncertain as this test continuously fails in automation (IE) due to selenium issues. I found it by testing manually today when checking for any issues that might be hiding.
The issue with alignment is because the 'org.icefaces.ace.datatable.scroll.ie8like7' config parameter is true in this test, but the visual testing done by Cruz is not in the intended IE8 IE7 rendering mode. As this parameter is part of a series of border case optimizations made for Williams it should not be turned on in our main test suite.
Unless the back issue is notable enough to reopen this issue it is complete.
Show
Nils Lundquist
added a comment - - edited The issue with alignment is because the 'org.icefaces.ace.datatable.scroll.ie8like7' config parameter is true in this test, but the visual testing done by Cruz is not in the intended IE8 IE7 rendering mode. As this parameter is part of a series of border case optimizations made for Williams it should not be turned on in our main test suite.
Unless the back issue is notable enough to reopen this issue it is complete.
| Field | Original Value | New Value |
|---|---|---|
| Reporter | Migration [ remote ] | Cruz Miraback [ cmiraback ] |
| Attachment | DataTable Styling Issue IE8.png [ 14889 ] |
| Assignee | Nils Lundquist [ nils.lundquist ] | |
| Fix Version/s | 3.2 [ 10338 ] | |
| Affects Version/s | 3.2 [ 10338 ] | |
| Assignee Priority | P1 [ 10010 ] |
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Resolution | Invalid [ 6 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
| Assignee Priority | P1 [ 10010 ] |
Interestingly this styling error doesn't occur in showcase.
For the second issue- is there any reason we care? This doesn't cause any errors. I can't help but think this is a separate bridge issue. At what revision was this regression first introduced?