Opened 14 years ago

Closed 13 years ago

Last modified 13 years ago

#13948 closed (fixed)

The sort arrows on the UI for the admin list display are backwards

Reported by: Jim Dalton Owned by: nobody
Component: contrib.admin Version: dev
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The sort arrows on the column headers of the admin change list view are backwards (i.e. inverted).

Right now, on the change list view, if you are sorting a column ascending, the arrow shown is a down-pointing triangle. If you sort a column descending, it is an up-pointing triangle.

This is counter-intuitive, and, I would argue, just plain wrong. Here are my reasons in case you need them:

My apologies if this has been covered or discussed elsewhere (my search turned up nothing) or if I am overlooking something ridiculous.

Attachments (1)

13948_changelist_arrows.diff (649 bytes ) - added by Julien Phalip 13 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 by Simon Meers, 14 years ago

Sounds like a sensible argument to me. Though perhaps the arrow is intended to indicate what the link will do if you click it, not how the list is currently sorted? Any UI experts want to advise?

comment:2 by Luke Plant, 14 years ago

I'm also convinced by this. The icon only appears for the column that is currently sorted, so you can't really argue that it shows what will happen if you click it.

comment:3 by Daishiman, 13 years ago

Triage Stage: UnreviewedDesign decision needed

Agreed. Some quick testing shows that the UI default is backwards. I spoke to Alex and he suggested to change it as DDN. Will ping jezdez so he can see can take a look at it.

comment:4 by Jannis Leidel, 13 years ago

Triage Stage: Design decision neededAccepted

by Julien Phalip, 13 years ago

comment:5 by Julien Phalip, 13 years ago

milestone: 1.3
Triage Stage: AcceptedReady for checkin

Hopefully the patch is trivial enough to allow me to RFC it.

comment:6 by Russell Keith-Magee, 13 years ago

Resolution: fixed
Status: newclosed

In [15495]:

Fixed #13948 -- Corrected the direction of arrows on the admin changelist. Thanks to jsdalton for the report, and Julien Phalip for the patch.

comment:7 by Russell Keith-Magee, 13 years ago

In [15497]:

[1.2.X] Fixed #13948 -- Corrected the direction of arrows on the admin changelist. Thanks to jsdalton for the report, and Julien Phalip for the patch.

Backport of r15495 from trunk.

comment:8 by Jacob, 13 years ago

milestone: 1.3

Milestone 1.3 deleted

Note: See TracTickets for help on using tickets.
Back to Top