Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#13948 closed (fixed)

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

Reported by: jsdalton 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:

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

Download all attachments as: .zip

Change History (9)

comment:1 Changed 5 years ago by DrMeers

  • 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 5 years ago by lukeplant

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

  • Triage Stage changed from Unreviewed to Design 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 5 years ago by jezdez

  • Triage Stage changed from Design decision needed to Accepted

Changed 5 years ago by julien

comment:5 Changed 5 years ago by julien

  • milestone set to 1.3
  • Triage Stage changed from Accepted to Ready for checkin

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

comment:6 Changed 5 years ago by russellm

  • Resolution set to fixed
  • Status changed from new to closed

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 5 years ago by russellm

In [15497]:

(The changeset message doesn't reference this ticket)

comment:8 Changed 4 years ago by jacob

  • milestone 1.3 deleted

Milestone 1.3 deleted

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