[FIXED-955] ListView column header - missing column header sort cue in gray theme

[icflorescu]

Ext version tested:

• Ext 3.3 rev 0
• Ext 3.2 rev 1

Adapter used:

• ext

css used:

• ext-all-notheme.css + xtheme-gray.css

Browser versions tested against:

• Internet Explorer 8.0
• Firefox 3.6.10 (firebug 1.5.4 installed)
• Chrome 6.0.472.63
• Safari 5.0.2

Operating System:

• Windows 7 Ultimate
• Mac OS X 10.6.4

Description:

• Missing sort direction arrow in gray theme on ListView column header.

Test Case: - see below

See this URL : http://dev.sencha.com/deploy/dev/exa...mes/index.html


Steps to reproduce the problem:

• access Ext JS Themes Example
• make sure default Blue Theme is selected
• go to Simple ListView on the bottom of the page and sort the ListView on any column; notice the arrow indicating the sorted column/direction
• switch to Gray Theme
• go to Simple ListView on the bottom of the page and sort the ListView on any column; notice there's no arrow (nor any other visual cue) indicating the sorted column/direction

The result that was expected:

• an arrow indicating the sort direction on the sorted column

The result that occurs instead:

• no visual cue to indicate the sort direction; there's no way of knowing which column/direction was sorted on

Screenshot or Video:

• attached ListView_in_Gray_Theme.png

Debugging already done:

• none

Possible fix:

Add the necessary image sprite to gray theme images folder and modify the corresponding css rule in xtheme-gray.css:

.x-list-header-inner em.sort-asc, .x-list-header-inner em.sort-desc {
    background-image: none;
    border-color: #d0d0d0;
}

In xtheme-blue.css, the corresponding rule links to a background image:

.x-list-header-inner em.sort-asc, .x-list-header-inner em.sort-desc {
  background-image: url(../images/default/grid/sort-hd.gif);
  border-color: #99BBE8;
}

If this behavior is by design, I suggest it should be changed, as right now there's no proper visual feedback on the sort column/direction.

[icflorescu]

Oops, I just noticed this is a duplicate of [OPEN-955]...

[meroy]

Marking this as Dupe: http://www.extjs.com/forum/showthrea...t-theme-sample

[meroy]

This has been fixed in SVN for 3.3.x revision 7252/7253.