Skip to content

applets/colorpicker: fix RadioButton

Christoph Wolk requested to merge cwo/kdeplasma-addons:colorpicker_settings into master

The RadioButton that allows choosing what the keyboard shortcut does can't actually be changed. There's no handler setting the config value on the second button, and while it seems like the onToggled handler on the first button might do the trick, it actually doesn't as onToggled only applies when being interactively flipped.

Instead just set it explicitly.

Test plan:

  1. Open color picker config, try to change the radio button that controls what the shortcut does
  2. (without) can't apply (with) can apply
Edited by Christoph Wolk

Merge request reports