Bug 483438

Summary: Timezones in Digital Clock widget appear selectable when they are not
Product: [Plasma] plasmashell Reporter: Mateus Melchiades <mateusbmelchiades>
Component: Digital Clock widgetAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: nate
Priority: NOR Keywords: qt6
Version First Reported In: 6.0.0   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 6.0.3
Sentry Crash Report:
Attachments: Current behavior with multiple timezones in Digital Clock

Description Mateus Melchiades 2024-03-13 13:04:46 UTC
Created attachment 167097 [details]
Current behavior with multiple timezones in Digital Clock

SUMMARY
When displaying multiple timezones in the Digital Clock widget, the first timezone in the list appears if it is "selected" but clicking it (or any other timezone) does nothing. Similarly, the user can also use the arrow keys to navigate the timezone list but selecting any of them with Enter also doesn't result in anything.
I think the ideal solution would be to either make the timezones not selectable or make it so selecting them opens KClock (maybe that's already the case and it's not working because I'm using the Flatpak?)

STEPS TO REPRODUCE
1. Configure Digital Clock to show multiple timezones
2. Open the widget
3. Observe timezones 

OBSERVED RESULT
Timezones are a selectable list but selecting them doesn't do anything.

EXPECTED RESULT
Either timezones should not be selectable or selecting them should result in some action (like opening KClock or displaying some message if it is not installed). Still, having the first timezone in the list as "selected" by default can cause confusion as it doesn't mean that it is the currently displayed timezone.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora Kinoite 40
KDE Plasma Version: 6.0.0
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2024-03-13 21:14:50 UTC
They're highlightable so you can focus them with a screen reader for accessibility purposes, but this probably only makes sense to do with a keyboard. I'll disable it with a pointing device.
Comment 2 Bug Janitor Service 2024-03-13 21:16:55 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/4064
Comment 3 Nate Graham 2024-03-14 15:12:37 UTC
Git commit 40257282d5168ba7bd88ea613fb0cafb0eecf613 by Nate Graham.
Committed on 14/03/2024 at 14:59.
Pushed by ngraham into branch 'master'.

applets/digital-clock: remove timezone highlight on click

It makes sense to select them with a keyboard for accessibility/screen
reader purposes, but not with a pointing device. Accordingly, disable
the visual change that appears on click.
FIXED-IN: 6.0.3

M  +3    -0    applets/digital-clock/package/contents/ui/CalendarView.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/40257282d5168ba7bd88ea613fb0cafb0eecf613
Comment 4 Nate Graham 2024-03-14 19:41:24 UTC
Git commit 676e93d8ef63c599d34da183cbb83e65b41f0012 by Nate Graham.
Committed on 14/03/2024 at 19:23.
Pushed by ngraham into branch 'Plasma/6.0'.

applets/digital-clock: remove timezone highlight on click

It makes sense to select them with a keyboard for accessibility/screen
reader purposes, but not with a pointing device. Accordingly, disable
the visual change that appears on click.
FIXED-IN: 6.0.3


(cherry picked from commit 40257282d5168ba7bd88ea613fb0cafb0eecf613)

M  +3    -0    applets/digital-clock/package/contents/ui/CalendarView.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/676e93d8ef63c599d34da183cbb83e65b41f0012