Mark KCM only as dirty when config should be saved (!17) · Merge requests · Plasma / Plasma Desktop

The source project of this merge request has been removed.

Alexander Lohnau requested to merge (removed):runner_reload_event into master May 27, 2020

This was introduced with https://phabricator.kde.org/D26445.

Connect configCommitted to ensure "Apply" gets enabled when you change any of the runner plugin settings.

I disagree with that. The saving of the individual runner KCMs is independent of the saving of the plugin selector.

The current behavior also suggests that you have to save the runners KCM and also the plugin selector for the config to be persisted, which is not true.

Also if you disable and then enable the plugin you can't click apply, because the changed parameter of the changed event is set to false. This parameter was ignored when the markAsConst method was used.

Edited May 30, 2020 by Alexander Lohnau

Mark KCM only as dirty when config should be saved

Merge request reports