D5411: restore the "current" color scheme concept
Marco Martin
noreply at phabricator.kde.org
Thu Apr 20 13:48:52 UTC 2017
mart added inline comments.
INLINE COMMENTS
> davidedmundson wrote in colorscm.cpp:134
> What is this about?
> blocking signals is a sign of something else being seriously wrong.
this is just copied from the old revision before the current entry was removed...
it can be safely removed as i prefer anyways that it highlights the actual current theme (like breeze) instead of the fake entry named "current"
> davidedmundson wrote in colorscm.cpp:458-459
> If the current design is for saving to be handled by signaling from the dialog to the KCM which does the actual saving. You should handle apply in the same way.
no, the saving is handled completely in the editor dialog.
this reloads the tiny previews, that is anyways a good thing to do in case of applied as well
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D5411
To: mart, #plasma
Cc: davidedmundson, plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170420/426c9550/attachment.html>
More information about the Plasma-devel
mailing list