D18519: [KColorSchemeEditor] Turn Save button into Save As
Kai Uwe Broulik
noreply at phabricator.kde.org
Fri Jan 25 11:49:59 GMT 2019
broulik created this revision.
broulik added reviewers: Plasma, VDG.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
The button effectively saves the theme to be edited as a new file, so turn it into a "Save As" button that is always shown.
TEST PLAN
Editing a system color scheme that you cannot overwrite
F6567497: Screenshot_20190125_124805.png <https://phabricator.kde.org/F6567497>
Editing a user color scheme, "Save As" allows to save it as a new theme, Apply just applies the changes to the edited theme
F6567498: Screenshot_20190125_124815.png <https://phabricator.kde.org/F6567498>
The Apply button is only shown when the `--overwrite` option is passed to kcolorscheme, passed by the KCM for user themes. I just figured, perhaps it makes more sense to check file permissions (it only does that once you actually hit Save) and show the Apply button based on that
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D18519
AFFECTED FILES
kcms/colors/editor/kcolorschemeeditor.cpp
kcms/colors/editor/scmeditordialog.cpp
kcms/colors/editor/scmeditordialog.h
To: broulik, #plasma, #vdg
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190125/2e0cccb5/attachment-0001.html>
More information about the Plasma-devel
mailing list