D12459: WIP: [Icon KCM] Port to new design
Kai Uwe Broulik
noreply at phabricator.kde.org
Sun Apr 22 20:44:33 UTC 2018
broulik created this revision.
broulik added reviewers: Plasma, VDG.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
This completely drops the icon customization with icon effects and only retains the icon size settings. I also couldn't find any trace of that "animations enabled" check box it used to have.
It adds drag and drop support for installing archives as themes. It can also download them from remote location when dropped from e.g. a browser. Installing theme files actually never worked in Plasma 5 as it got broken in the port from KIO NetAccess to storedGet.
Implements T7262 <https://phabricator.kde.org/T7262>
TEST PLAN
TODO:
- Figure out "Reset to defaults", it reverts to "hicolor" and then you end up with no icons. The old KCM just stored the "current" theme and resets to that
- Applying settings for kde4 apps
- Remove the old code, perhaps split out the icon effects stuff as separate expert tool?
F5818181: Screenshot_20180422_222955.png <https://phabricator.kde.org/F5818181>
Icon size config flyout, I chose not to use a separate page as in the mockup as it's really just the size settings now:
F5818182: Screenshot_20180422_223212.png <https://phabricator.kde.org/F5818182>
The slider shows the possible size steps
F5818183: Screenshot_20180422_223222.png <https://phabricator.kde.org/F5818183>
Removing installed theme
F5818184: Screenshot_20180422_223253.png <https://phabricator.kde.org/F5818184>
It comes with an undo feature and only if you hit Apply or OK it will acutally delete the themes, so you can undo individual or all (by using Revert button) deletions. If this is accepted, I'll look into rolling this out to the other KCMs that allow permanently deleting actual files. It is inspired by wallpaper settings.
F5818185: Screenshot_20180422_223325.png <https://phabricator.kde.org/F5818185>
Installing theme with `InlineMessage`
F5818186: Screenshot_20180422_223348.png <https://phabricator.kde.org/F5818186>
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D12459
AFFECTED FILES
kcms/icons/CMakeLists.txt
kcms/icons/Messages.sh
kcms/icons/icons.desktop
kcms/icons/kcm_icons.desktop
kcms/icons/main.cpp
kcms/icons/main.h
kcms/icons/package/contents/ui/IconSizePopup.qml
kcms/icons/package/contents/ui/main.qml
kcms/icons/package/metadata.desktop
kcms/icons/tests/CMakeLists.txt
kcms/icons/tests/testicons.cpp
To: broulik, #plasma, #vdg
Cc: plasma-devel, 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/20180422/6fa69671/attachment.html>
More information about the Plasma-devel
mailing list