https://bugs.kde.org/show_bug.cgi?id=500702

cwo <cwo....@posteo.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/powerdevil/-/commit/6116 |ma/powerdevil/-/commit/2ac9
                   |99d9aae7061b96e306370728271 |69859626964802cf63300baec54
                   |277334de5                   |f036f9425

--- Comment #4 from cwo <cwo....@posteo.net> ---
Git commit 2ac969859626964802cf63300baec54f036f9425 by Christoph Wolk.
Committed on 25/02/2025 at 10:35.
Pushed by cwo into branch 'Plasma/6.3'.

applets/brightness: fix Return press on switch

The "Suspend" switch for Night Light calls toggle() on space/enter/
return press, and handles the onToggled() signal to actually perform the
action. The qqc2 api is exceptionally confusing here, toggle() refers to
changing the checked state of a button, but does not automatically
call toggled() which is called when a user interactively manipulates a
control, and would need to be called explicitly in the handler. The only
reason it worked for Space is that the event wasn't accepted, so the
switch called toggled() internally.

Instead, just use clicked() and avoid the whole messy toggled/toggle()
situation. Along the way, we also explicitly accept the onPressed events
when we handle them, to avoid problems with multiple activation in the
future.
FIXED-IN: 6.3.2

M  +5    -3    applets/brightness/package/contents/ui/NightLightItem.qml

https://invent.kde.org/plasma/powerdevil/-/commit/2ac969859626964802cf63300baec54f036f9425

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to