Skip to content
This repository was archived by the owner on Nov 17, 2020. It is now read-only.

Fix backwards sort arrow icon. #88

Merged
merged 1 commit into from
Nov 27, 2015
Merged

Fix backwards sort arrow icon. #88

merged 1 commit into from
Nov 27, 2015

Conversation

serhalp
Copy link
Contributor

@serhalp serhalp commented Nov 26, 2015

It's conventional in GUIs to use a triangle or arrow icon to indicate how data is currently sorted in a column – where the wide part of the triangle being at the top and the narrow part at the bottom indicates that larger rows are above (decreasing sort), and vice versa. In other words, the direction in which the icon width increases reflects the direction in which the values in the column increase.

The current icon is contrary to most UIs out there, which can be misleading – and it just feels plain wrong:

screen shot 2015-11-26 at 3 43 57 pm

This PR swaps the two icons.

It's conventional in GUIs to use a triangle or arrow icon to indicate how
data is currently sorted in a column - where the wide part of the triangle being
at the top and the narrow part at the bottom indicates that larger rows are above
(decreasing sort), and vice versa.  In other words, the direction in which the
icon width increases reflects the direction in which the values in the column increase.
@michaelklishin
Copy link
Contributor

@serhalp good catch, thank you!

michaelklishin added a commit that referenced this pull request Nov 27, 2015
Fix backwards sort arrow icon.
@michaelklishin michaelklishin merged commit d1c5387 into rabbitmq:stable Nov 27, 2015
@michaelklishin michaelklishin self-assigned this Nov 27, 2015
@michaelklishin michaelklishin modified the milestones: 3.6.0, 3.5.7 Nov 27, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants