[Colors KCM] Correctly report whether saving is needed
ClosedPublic

Authored by davidre on Jan 8 2020, 3:52 PM.

Details

Summary

Previously the kcm always reported that a save is needed once a scheme had been
marked for deletion. The correct way is to check if there are schemes that are
currently marked for deletion. Also we don't need the firstTime workaround.

Test Plan

Mark a scheme for deletion and undo that action

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidre created this revision.Jan 8 2020, 3:52 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 8 2020, 3:52 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidre requested review of this revision.Jan 8 2020, 3:52 PM
davidre edited the summary of this revision. (Show Details)Jan 8 2020, 6:51 PM
mart accepted this revision.Jan 10 2020, 12:01 PM
This revision is now accepted and ready to land.Jan 10 2020, 12:01 PM
This revision was automatically updated to reflect the committed changes.