ICEfaces
  1. ICEfaces
  2. ICE-10650

showcase ace:dataTable > MultiRow Header style issue with IE 7

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P02
    • Fix Version/s: EE-3.3.0.GA_P03
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces EE-3.3.0 maintenance branch r44538. Tomcat 7, IE 7, showcase ace:dataTable > MultiRow Header demo
    • Assignee Priority:
      P2

      Description

      showcase ace:dataTable > MultiRow Header
      Style issue when demo loads, see screenshot dt.PNG

      Interacting with demo causes further layout issues, see screenshot dt2.PNG
      1. dt.PNG
        25 kB
      2. dt2.PNG
        29 kB

        Activity

        Hide
        Arturo Zambrano added a comment -

        The markup looks correct. It seems like the rowSpan atttribute of the <table> element is not being respected in IE7. Do we know if this was working before?

        Show
        Arturo Zambrano added a comment - The markup looks correct. It seems like the rowSpan atttribute of the <table> element is not being respected in IE7. Do we know if this was working before?
        Hide
        Arturo Zambrano added a comment - - edited

        This issue only happens when there's either row expansion or a panel expansion. The reason is that a hidden <tr> element is rendered as a placeholder for the expansion row, and this <tr> is accounted for towards the rowSpan value on IE7. In other browsers, when a <tr> is hidden, it doesn't count towards this value, and the rows are displayed correctly, but on IE7 they still count, so table cells with rowSpan values greater than 1, won't span all the visible rows indicated by the attribute.

        Show
        Arturo Zambrano added a comment - - edited This issue only happens when there's either row expansion or a panel expansion. The reason is that a hidden <tr> element is rendered as a placeholder for the expansion row, and this <tr> is accounted for towards the rowSpan value on IE7. In other browsers, when a <tr> is hidden, it doesn't count towards this value, and the rows are displayed correctly, but on IE7 they still count, so table cells with rowSpan values greater than 1, won't span all the visible rows indicated by the attribute.
        Hide
        Arturo Zambrano added a comment -

        r44582: committed fix to calculate appropriate rowSpan values on IE7 and apply them on the client, when the table has grouping and expansion enabled.

        Show
        Arturo Zambrano added a comment - r44582: committed fix to calculate appropriate rowSpan values on IE7 and apply them on the client, when the table has grouping and expansion enabled.
        Hide
        Arturo Zambrano added a comment -

        r44592: backported fix to tag.

        Show
        Arturo Zambrano added a comment - r44592: backported fix to tag.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces EE-3.3.0.GA_P03 tag, ICEfaces EE-3.3.0 maintenance branch r44593. Tomcat 7, IE 7.

        Show
        Liana Munroe added a comment - Verified ICEfaces EE-3.3.0.GA_P03 tag, ICEfaces EE-3.3.0 maintenance branch r44593. Tomcat 7, IE 7.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces EE-3.3.0.GA_P03 Jenkins build 02, Tomcat 7, IE 7.

        Show
        Liana Munroe added a comment - Verified ICEfaces EE-3.3.0.GA_P03 Jenkins build 02, Tomcat 7, IE 7.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: