Opened 6 years ago

Closed 6 years ago

Last modified 5 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: master
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


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 6 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 6 years ago by Simon Meers

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

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 Changed 6 years ago by Luke Plant

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 Changed 6 years ago by Daishiman

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 Changed 6 years ago by Jannis Leidel

Triage Stage: Design decision neededAccepted

Changed 6 years ago by Julien Phalip

comment:5 Changed 6 years ago by Julien Phalip

milestone: 1.3
Triage Stage: AcceptedReady for checkin

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

comment:6 Changed 6 years ago by Russell Keith-Magee

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 Changed 6 years ago by Russell Keith-Magee

In [15497]:

(The changeset message doesn't reference this ticket)

comment:8 Changed 5 years ago by Jacob

milestone: 1.3

Milestone 1.3 deleted

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