<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/D12278">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Pretty much WIP<br />
What works:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Grid layout with crappy thumbnail<ul class="remarkup-list">
<li class="remarkup-list-item">The screenshots just show a quick and dirty attempt to mimic the old tiny icons we used to have. Perhaps either it should show *all* available colors in the theme but that might turn into a pixel mudd? or instead show a live preview of some controls, maybe a "fake window" in there?</li>
</ul></li>
<li class="remarkup-list-item">Downloading color themes through GHNS</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>Implements <a href="https://phabricator.kde.org/T7243" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T7243</a></p>

<p>Not too fond of this "Current" (carried over from the old code). Perhaps we could do some magic to only show it when the current theme is not in the list or is a modified version of a theme? That would make the dialog less confusing as to what "Current" means.</p>

<p><a href="https://phabricator.kde.org/F5810572" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F5810572: Screenshot_20180417_132441.png</a><br />
TODO:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Make applying theme work (write config and signal applications)</li>
<li class="remarkup-list-item">Make restore defaults work</li>
<li class="remarkup-list-item">Make "Install from file" work</li>
<li class="remarkup-list-item">Make edit work (the edit dialog is quite entangled with the code, I thought it was completely standalone nowadays)</li>
<li class="remarkup-list-item">Make removing theme work</li>
<li class="remarkup-list-item">Make the "apply to non Qt apps" checkbox work (no idea why anyone would want to uncheck this, though)</li>
</ul></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/D12278">https://phabricator.kde.org/D12278</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/colors/CMakeLists.txt<br />
kcms/colors/colors.cpp<br />
kcms/colors/colors.desktop<br />
kcms/colors/colors.h<br />
kcms/colors/colorscm.cpp<br />
kcms/colors/kcm_colors.desktop<br />
kcms/colors/package/contents/ui/main.qml<br />
kcms/colors/package/metadata.desktop</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>