<table><tr><td style="">broulik updated this revision to Diff 32881.<br />broulik edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-5nby67whmvfrfxe/">(Show Details)</a><br />broulik edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-k57exv2lvmobo7q/">(Show Details)</a><br />broulik added a comment.
</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><div><ul class="remarkup-list">
<li class="remarkup-list-item">Implement applying color schemes and notifying applications about that</li>
<li class="remarkup-list-item">Implement "Install from file" including drag and drop</li>
<li class="remarkup-list-item">Implement removing color schemes including undo feature</li>
<li class="remarkup-list-item">Remove "apply to non qt applications" checkbox</li>
<li class="remarkup-list-item">Somewhat improve thumbnail but it's still not ideal</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Pretty much WIP<br />
What works:<br />
* Grid layout with crappy thumbnail<br />
** 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?</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">This removes the "Apply to non-Qt applications checkbox" which isn't something one should uncheck.</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Downloading color</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">It is still read from config file and honored but</span> the<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">mes through GHNS</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">re's no in-your-face checkbox for that anymore.</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Closes T7243<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">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.<br />
<br />
{F5810572}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Revision 1</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">TODO:</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F5810572}</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Make applying theme work (write config and signal applications)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Revision 2</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Make restore defaults work<br />
* Make "Install from file" work</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F5819205}<br />
<br />
TODO:</span><br />
* Make <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">edit work (the edit dialog is quite entangled with the code, I thought it was completely standalone nowadays)</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">restore defaults work</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">* Make removing</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">* Figure out the "Current"</span> theme <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">work</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">stuff</span><br />
* Make <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the "apply to non Qt apps" checkbox work (no idea why anyone would want to uncheck this</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">edit work (the edit dialog is quite entangled with the code</span>, <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">I </span>though<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">)<br />
</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">t it was completely standalone nowadays)</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D12278?vs=32376&id=32881">https://phabricator.kde.org/D12278?vs=32376&id=32881</a></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/Messages.sh<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>mart, abetts, ngraham, davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol<br /></div>