ICEfaces
  1. ICEfaces
  2. ICE-4988

Sortable dataTable header (commandSortHeader) not aligned correctly in IE8

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 1.8
          • Fix Version/s: 1.8.2-EE-Beta, 1.8.2-EE-GA, 1.8.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            dataTable, commandSortHeader + IE8
          • ICEsoft Forum Reference:
          • Affects:
            Sample App./Tutorial
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            IE8 working in IE7-compatibility mode displays the headers correctly. Therefore, a valid workaround is placing the snippet below in your pages (or just in the mater page if you leverage facelets templating features):

            <meta http-equiv="X-UA-Compatible" content="IE=7" />
            Show
            IE8 working in IE7-compatibility mode displays the headers correctly. Therefore, a valid workaround is placing the snippet below in your pages (or just in the mater page if you leverage facelets templating features): <meta http-equiv="X-UA-Compatible" content="IE=7" />

            Description

            The commandSortHeader doesn't align correctly with the dataTable columns when viewed in IE8 (native mode).
            1. screenshot-1.png
              223 kB
            2. screenshot-2.png
              242 kB
            3. screenshot-3.png
              223 kB
            4. screenshot-4.png
              231 kB

              Activity

              Ascending order - Click to sort in descending order
              Adnan Durrani created issue -
              Hide
              Permalink
              Adnan Durrani added a comment -

              I have tested it with 1.8.2, 1.8.1 and 1.8.0 and can reproduce this. Seems like its there for a while.

              Show
              Adnan Durrani added a comment - I have tested it with 1.8.2, 1.8.1 and 1.8.0 and can reproduce this. Seems like its there for a while.
              Adnan Durrani made changes -
              Field Original Value New Value
              Assignee Ken Fyten [ ken.fyten ]
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 1.8.3 [ 10211 ]
              Affects Version/s 1.8 [ 10161 ]
              Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
              Priority Major [ 3 ] Minor [ 4 ]
              Ken Fyten made changes -
              Summary Sortable header not aligned in IE8 Sortable dataTable header (commandSortHeader) not aligned correctly in IE8
              Workaround Description IE8 working in IE7-compatibility mode displays the headers correctly. Therefore, a valid workaround is placing the snippet below in your pages (or just in the mater page if you leverage facelets templating features):

              <meta http-equiv="X-UA-Compatible" content="IE=7" />
              Environment Sortable header + aligned + IE8 dataTable, commandSortHeader + IE8
              Workaround Exists [Yes]
              Salesforce Case []
              Affects [Compatibility/Configuration]
              Description Sortable header not aligned in IE8 The commandSortHeader doesn't align correctly with the dataTable columns when viewed in IE8 (native mode).
              Hide
              Permalink
              Ken Fyten added a comment -

              Will review with QA why this hasn't been caught before.

              Show
              Ken Fyten added a comment - Will review with QA why this hasn't been caught before.
              yip.ng made changes -
              Attachment screenshot-1.png [ 12026 ]
              yip.ng made changes -
              Attachment screenshot-2.png [ 12027 ]
              Hide
              Permalink
              yip.ng added a comment -

              Seems this is caused by our use of table within span within anchor. See screenshots 1 and 2.

              Show
              yip.ng added a comment - Seems this is caused by our use of table within span within anchor. See screenshots 1 and 2.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #19424 Thu Oct 15 10:23:00 MDT 2009 yip.ng ICE-4988: Fixed sortable data table header alignment problems.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp-portlet.css
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/xp/xp.css
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/woodstock/woodstock.css
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/royale/royale.css
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/rime.css
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/resources/css/rime/rime-portlet.css
              yip.ng made changes -
              Attachment screenshot-3.png [ 12041 ]
              Hide
              Permalink
              yip.ng added a comment - - edited

              Changed CSS to display table like an inline element.

              Fixed. See screen shot 3. Also tested on IE 6 and FF 3.5.

              Show
              yip.ng added a comment - - edited Changed CSS to display table like an inline element. Fixed. See screen shot 3. Also tested on IE 6 and FF 3.5.
              yip.ng made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Joanne Bai added a comment -

              Sortable Header layout issue is fixed in component showcase jsp and facelets version, but not facelets-enh version

              Show
              Joanne Bai added a comment - Sortable Header layout issue is fixed in component showcase jsp and facelets version, but not facelets-enh version
              Joanne Bai made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Affects [Compatibility/Configuration] [Sample App./Tutorial]
              Hide
              Permalink
              yip.ng added a comment -

              Only changed CSS. So now facelets-enh version has different interpretation of same CSS? How can that be?

              Show
              yip.ng added a comment - Only changed CSS. So now facelets-enh version has different interpretation of same CSS? How can that be?
              yip.ng made changes -
              Attachment screenshot-4.png [ 12052 ]
              Hide
              Permalink
              yip.ng added a comment -

              Can't reproduce problem in facelets-enh version. See screenshot-4.

              Show
              yip.ng added a comment - Can't reproduce problem in facelets-enh version. See screenshot-4.
              Hide
              Permalink
              Joanne Bai added a comment -

              Not reproducible either on ICEfaces trunk revision 19443. Fix confirmed on IE8.

              Show
              Joanne Bai added a comment - Not reproducible either on ICEfaces trunk revision 19443. Fix confirmed on IE8.
              Joanne Bai made changes -
              Status Reopened [ 4 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Closed [ 6 ] Reopened [ 4 ]
              Assignee Yip Ng [ yip.ng ]
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 1.8.2-EE-Beta [ 10215 ]
              Ken Fyten made changes -
              Fix Version/s 1.8.2-EE-GA [ 10216 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Adnan Durrani
                • Votes:
                  1 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: