PDA

View Full Version : [FIXED-955] ListView column header - missing column header sort cue in gray theme



icflorescu
12 Oct 2010, 1:02 PM
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/examples/themes/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
12 Oct 2010, 1:06 PM
Oops, I just noticed this is a duplicate of [OPEN-955]...

meroy
15 Oct 2010, 12:31 PM
Marking this as Dupe: http://www.extjs.com/forum/showthread.php?99129-Sort-indicator-icon-is-missing-for-the-Simple-ListView-quot-Gray-quot-theme-sample

meroy
3 Nov 2010, 4:46 PM
This has been fixed in SVN for 3.3.x revision 7252/7253.