Bug #11127

sort direction indicator in list module incorrect

Added by Joh. Feustel over 2 years ago. Updated 12 months ago.

Status:Closed Start date:2010-11-29
Priority:Could have Due date:
Assignee:- % Done:

0%
Category:Technical
Target version:TYPO3 6.0
Tags:
Votes: 0

Description

In the list module the sort direction indicator icon in the table header is wrong. Arrow up stands for desc which is not common. Mac Finder, Windows Explorer, phpMyAdmin and also the recycler module does it the other way round. See attached screenshots.
I think we should change this to be consistent. Also we should think of replacing the redup.gif with arrowup.png
What do you think?
BTW: should be a sprite icon

Cheers,
Johannes


sorting_list_module.jpg (23.6 kB) Joh. Feustel, 2010-11-29 11:04

sorting_recycler.jpg (20.9 kB) Joh. Feustel, 2010-11-29 11:04

file-list1.png (32.3 kB) Helmut Hummel, 2012-07-07 17:13

file-list2.png (45.4 kB) Helmut Hummel, 2012-07-07 17:13

recycler.png (15.4 kB) Helmut Hummel, 2012-07-07 17:13

list-module.png (34.8 kB) Helmut Hummel, 2012-07-07 17:13

Table_Header_Gradient.png (65.7 kB) Jens Hoffmann, 2012-07-09 13:27

Associated revisions

Revision f53b572e
Added by Helmut Hummel 12 months ago

[BUGFIX] Use lighter sorting icon on dark background

Change f28bdc1 introduced new sorting icons for the list
and file list module. To further improve it, use light
arrows for the dark background in the file list module.

Additionally change the naming from "sort" to "sorting"

Change-Id: I5132e684ce461c61be097608f2d90f36f05528b9
Resolves: #11127
Releases: 6.0
Reviewed-on: http://review.typo3.org/12692
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter

History

#1
Updated by Jens Hoffmann 12 months ago

  • Category set to Technical
  • Status changed from New to Under Review
  • Priority changed from Should have to Could have
  • Target version set to TYPO3 6.0

Thanks for the report - sorry for the late answer.
Technical thing - but it should be fixed!

#2
Updated by Wouter Wolters 12 months ago

Will try to take care of this one.

#3
Updated by Gerrit Code Review 12 months ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12688

#4
Updated by Gerrit Code Review 12 months ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12688

#5
Updated by Gerrit Code Review 12 months ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12688

#6
Updated by Gerrit Code Review 12 months ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12688

#7
Updated by Gerrit Code Review 12 months ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12692

#8
Updated by Helmut Hummel 12 months ago




#9
Updated by Jens Hoffmann 12 months ago

  • Status changed from Under Review to Closed

THX :) Done.

Closed.

#10
Updated by Gerrit Code Review 12 months ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12692

#11
Updated by Jens Hoffmann 12 months ago

I would say we should also adjust the Table header to be streamlined with all other Modules.
So the gradient ist missing here.

A possible solution could be:

Also available in: Atom PDF