ICEfaces
  1. ICEfaces
  2. ICE-9319

ace:dataTable - ice-8237 - header cell sorted style

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 4.0.BETA
          • Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            *
          • Assignee Priority:
            P1

            Description

            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.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Nils Lundquist added a comment -

              This issue existed in 3.3. Can be seen in the screenshot in ICE-9317

              Show
              Nils Lundquist added a comment - This issue existed in 3.3. Can be seen in the screenshot in ICE-9317
              Hide
              Permalink
              Nils Lundquist added a comment -

              This style has always been this way.

              This is due to a constraint of the JQuery theme being used. The background gradient is of a fixed height, but the header cell exceeds that height.

              The test could be redesigned with smaller fonts / less header cell padding / less text to prevent this issue.

              This issue can also be worked around at the app level by overriding that header cell background rule with one more specific.

              Show
              Nils Lundquist added a comment - This style has always been this way. This is due to a constraint of the JQuery theme being used. The background gradient is of a fixed height, but the header cell exceeds that height. The test could be redesigned with smaller fonts / less header cell padding / less text to prevent this issue. This issue can also be worked around at the app level by overriding that header cell background rule with one more specific.
              Hide
              Permalink
              Ken Fyten added a comment -

              Let's make a new JIRA to alter the Sam theme to use gradients or flat background colour (on browsers that don't support gradients IE7?) that will fix this issue.

              Show
              Ken Fyten added a comment - Let's make a new JIRA to alter the Sam theme to use gradients or flat background colour (on browsers that don't support gradients IE7?) that will fix this issue.
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #34691
              Committed by nils.lundquist
              Moments ago
              ICE-9319 - Added header cell sorted style override to test case. Work around fixed height gradients.

              Show
              Nils Lundquist added a comment - Revision #34691 Committed by nils.lundquist Moments ago ICE-9319 - Added header cell sorted style override to test case. Work around fixed height gradients.
              Hide
              Permalink
              Ken Fyten added a comment -

              Missed committing this to the EE branch.

              Show
              Ken Fyten added a comment - Missed committing this to the EE branch.
              Hide
              Permalink
              Nils Lundquist added a comment -

              We need a fix for test case styling in the EE branch? I guess...

              Show
              Nils Lundquist added a comment - We need a fix for test case styling in the EE branch? I guess...
              Hide
              Permalink
              Nils Lundquist added a comment -

              This was an test change - no EE changes required.

              Show
              Nils Lundquist added a comment - This was an test change - no EE changes required.

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Nils Lundquist
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: