Skip to content

Just show active chevron for sorted table column - #1671

Merged
gregorydlogan merged 3 commits into
opencast:r/20.xfrom
geichelberger:sort-chevron
Sep 21, 2026
Merged

gregorydlogan merged 3 commits into
opencast:r/20.xfrom
geichelberger:sort-chevron

Conversation

@geichelberger

Copy link
Copy Markdown
Contributor

Only display the active chevron for sorted table columns. Previously, both chevrons were visible, with one highlighted when a column was sorted; now the opposite chevron is hidden, and the vertical offset only applies when both are displayed.

Only display the active chevron for sorted table columns. Previously,
both chevrons were visible, with one highlighted when a column was
sorted; now the opposite chevron is hidden, and the vertical offset only
applies when both are displayed.
@geichelberger geichelberger added type:enhancement New feature or request type:usability Improves the UX labels Sep 9, 2026
@Arnei

Arnei commented Sep 10, 2026

Copy link
Copy Markdown
Member

What issue is this trying to solve? Double chevrons are perfectly conventional as far as I'm aware.

@geichelberger

Copy link
Copy Markdown
Contributor Author

You need pretty good eyes to recognize the sort order. I just copied the approach our UX designers created for another tool.

image

vs

image

@Arnei

Arnei commented Sep 11, 2026

Copy link
Copy Markdown
Member

Gotcha. I'm not a fan of using single chevrons in this case. While it is a serviceable solution, it makes the ui appear inconsistent. Having one arrow where there were two before is confusing, especially when all other columns still display two arrows.

Personally I would first check if upping the contrast helps. If that is not enough we could also consider filled-in triangles instead of chevrons (https://react-icons.github.io/react-icons/search/#q=faCaretUp). And if single chevron is truly the only remaining option, we might want to consider removing double chevrons altogether.

@geichelberger

Copy link
Copy Markdown
Contributor Author

We could hide the double arrows and only show them on hover. Like https://mui.com/material-ui/react-table/

@geichelberger

Copy link
Copy Markdown
Contributor Author

Hidden chevron with accessibility support:

2026-09-15.13-30.Bildschirm.OC.Studio.webm

@gregorydlogan gregorydlogan self-assigned this Sep 21, 2026
@gregorydlogan
gregorydlogan merged commit 9d6cae4 into opencast:r/20.x Sep 21, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:enhancement New feature or request type:usability Improves the UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants