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

cwo <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/kdeplasma-addons/-/commi |ma/kdeplasma-addons/-/commi
                   |t/016448f4d3058bb02bd04e1f5 |t/1b2603d19a8585ae1439d2d51
                   |8197367f8b870b6             |74bbe0936bff90a

--- Comment #3 from cwo <[email protected]> ---
Git commit 1b2603d19a8585ae1439d2d5174bbe0936bff90a by Christoph Wolk.
Committed on 26/01/2026 at 19:09.
Pushed by cwo into branch 'Plasma/6.6'.

applets/colorpicker: always send currentColorChanged

GrabWidget returns early if the currentColor is changed to the same
color it already is. While this is usually a good idea, it's not here
– the color being set only happens when the user has completed picking
a color, and the widget needs to be informed about this. There's some
extra deduplicating logic there anyway, and re-picking the same color
is legitimate - the user may do this to copy it to the clipboard again,
or may have deleted it from the history, then reconsidered and now try
to re-add it by picking the same color again.

Instead, don't return early if unchanged, and always send the Changed
signal.
FIXED-IN: 6.6.0


(cherry picked from commit 016448f4d3058bb02bd04e1f58197367f8b870b6)

Co-authored-by: Christoph Wolk <[email protected]>

M  +2    -3    applets/colorpicker/grabwidget.cpp

https://invent.kde.org/plasma/kdeplasma-addons/-/commit/1b2603d19a8585ae1439d2d5174bbe0936bff90a

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

Reply via email to