<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, VDG.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12459">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>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.<br />
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.<br />
Implements <a href="https://phabricator.kde.org/T7262" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T7262</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><p>TODO:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">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</li>
<li class="remarkup-list-item">Applying settings for kde4 apps</li>
<li class="remarkup-list-item">Remove the old code, perhaps split out the icon effects stuff as separate expert tool?</li>
</ul>

<p><a href="https://phabricator.kde.org/F5818181" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5818181: Screenshot_20180422_222955.png</a><br />
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:<br />
<a href="https://phabricator.kde.org/F5818182" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5818182: Screenshot_20180422_223212.png</a><br />
The slider shows the possible size steps<br />
<a href="https://phabricator.kde.org/F5818183" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5818183: Screenshot_20180422_223222.png</a><br />
Removing installed theme<br />
<a href="https://phabricator.kde.org/F5818184" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5818184: Screenshot_20180422_223253.png</a><br />
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.<br />
<a href="https://phabricator.kde.org/F5818185" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5818185: Screenshot_20180422_223325.png</a><br />
Installing theme with <tt style="background: #ebebeb; font-size: 13px;">InlineMessage</tt><br />
<a href="https://phabricator.kde.org/F5818186" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5818186: Screenshot_20180422_223348.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12459">https://phabricator.kde.org/D12459</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/icons/CMakeLists.txt<br />
kcms/icons/Messages.sh<br />
kcms/icons/icons.desktop<br />
kcms/icons/kcm_icons.desktop<br />
kcms/icons/main.cpp<br />
kcms/icons/main.h<br />
kcms/icons/package/contents/ui/IconSizePopup.qml<br />
kcms/icons/package/contents/ui/main.qml<br />
kcms/icons/package/metadata.desktop<br />
kcms/icons/tests/CMakeLists.txt<br />
kcms/icons/tests/testicons.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>