Bug 500703 - Keyboard navigation to "Night Light Suspend" switch is unintuitive
Summary: Keyboard navigation to "Night Light Suspend" switch is unintuitive
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Brightness and Color widget (show other bugs)
Version: 6.3.1
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-02-24 23:38 UTC by nilskemail+kde
Modified: 2025-02-26 06:26 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.3.3
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description nilskemail+kde 2025-02-24 23:38:18 UTC
SUMMARY
Getting keyboard focus on the Night Light Suspend switch is hard because it is not reachable when tabbing forwards but only when tabbing backwards (SHIFT+TAB) for whatever reason?

STEPS TO REPRODUCE
1. Open the Brightness and Color widget
2. Press TAB multiple times
3. Press SHIFT+TAB multiple times

OBSERVED RESULT
The switch is never focused when tabbing forward but only when tabbing backwards

EXPECTED RESULT
The switch should be focused in both directions

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 41
KDE Plasma Version: 6.3.1
KDE Frameworks Version: 6.11.0
Qt Version: 6.8.2
Kernel Version: 6.12.15-200.fc41.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 16 × AMD Ryzen 7 7840U w/ Radeon  780M Graphics
Memory: 30.6 GiB of RAM
Graphics Processor: AMD Radeon 780M
Comment 1 Bug Janitor Service 2025-02-25 11:16:05 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/powerdevil/-/merge_requests/526
Comment 2 cwo 2025-02-26 06:26:33 UTC
Git commit a3ded3a98ce5c76a643d2f9d783537ecbb279ca2 by Christoph Wolk.
Committed on 26/02/2025 at 06:23.
Pushed by cwo into branch 'master'.

applets/brightness: don't skip NightLight switch on tab

The Night Light delegate has special handling to select the switch on
tab, but this is done in the delegate's contentItem, rather than
directly in the delegate. As a consequence, it only works if the
contentItem has focus, not if the delegate itself has focus, which
happens regularly.

Instead, put the handler in the delegate directly; if focus is on the
RowLayout (or elsewhere generally in the contentItem), handling will
fall back to the delegate (unless overridden) anyway so this is the only
place where we really need it.
FIXED-IN: 6.3.3

M  +1    -2    applets/brightness/package/contents/ui/NightLightItem.qml

https://invent.kde.org/plasma/powerdevil/-/commit/a3ded3a98ce5c76a643d2f9d783537ecbb279ca2