ICEfaces
  1. ICEfaces
  2. ICE-8634

ace:dataTable - Styling issue and JS error in IE

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major 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/ICE-8443.jsf
      /views/ICE-8104.jsf
      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/ICE-8524.jsf'
      3) Select a filter option from the list
      4) Select another
      Similar JS error is thrown

        Activity

        Migration created issue -
        Hide
        Nils Lundquist added a comment - - edited

        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?

        Show
        Nils Lundquist added a comment - - edited 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?
        Hide
        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.

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

        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.
        Migration made changes -
        Field Original Value New Value
        Reporter Migration [ remote ] Cruz Miraback [ cmiraback ]
        Migration made changes -
        Attachment DataTable Styling Issue IE8.png [ 14889 ]
        Migration made changes -
        Assignee Nils Lundquist [ nils.lundquist ]
        Fix Version/s 3.2 [ 10338 ]
        Affects Version/s 3.2 [ 10338 ]
        Assignee Priority P1 [ 10010 ]
        Migration made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Invalid [ 6 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P1 [ 10010 ]

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Cruz Miraback
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: