Bug 442234 - Sort menu dropdowns should use radio buttons
Summary: Sort menu dropdowns should use radio buttons
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (other bugs)
Version First Reported In: 21.08.1
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2021-09-09 16:34 UTC by Nagy Tibor
Modified: 2024-02-01 18:05 UTC (History)
3 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/qqc2-desktop-style/-/commit/9dadd05de66a135ea0c3c66c6bb554ab22ec6a67
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
Radio buttons on the Track sort menu using RadioDelegate and RadioButton (19.59 KB, image/png)
2022-01-04 18:24 UTC, Jack Hill 		Details
This menu in Discover (83.67 KB, image/png)
2022-01-10 18:16 UTC, Nagy Tibor 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nagy Tibor 2021-09-09 16:34:52 UTC 
SUMMARY
Currently the sort menu dropdowns at the page headers use checkbox controls. They are not appropriate there because sorting options are mutually exclusive options. They should be converted into radio buttons.

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.22
KDE Plasma Version: 5.22.5
KDE Frameworks Version: 5.85.0
Qt Version: 5.15.3
Comment 1 Jack Hill 2022-01-04 18:24:40 UTC 
Created attachment 145113 [details]
Radio buttons on the Track sort menu using RadioDelegate and RadioButton

This is doable by changing MenuItem to RadioButton/RadioDelegate in SortMenu.qml.
Unfortunately RadioButton loses the padding and RadioDelegate places the radio buttons on the right rather than on the left.
Comment 2 Nagy Tibor 2022-01-10 18:16:42 UTC 
Created attachment 145304 [details]
This menu in Discover

How does Discover implement this? It also has a similar menu that seems to work okayish and only has a minimally borked metrics.