<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, VDG.<br />Herald added a project: Plasma.<br />Herald 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/D24361">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This ports the widget style KCM to use a grid view KCM with live preview thumbnails and inline configure button.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Icon and tool bar settings have been moved into a popup, similar to Icons KCM, to leave more room for the rather large thumbnails.</li>
<li class="remarkup-list-item">Overall cleanup of unused/defunct features and modernized codebase</li>
<li class="remarkup-list-item">Support for non-Linux is dropped, i.e. it assumes "Breeze" as the default, no "native" (e.g. Mac or Windows style) as we want our apps to follow whatever other platform they run in, rather than offering this KCM there</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p><a href="https://phabricator.kde.org/F7496756" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7496756: Screenshot_20191002_135630.png</a><br />
"Advanced" settings<br />
<a href="https://phabricator.kde.org/F7496757" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7496757: Screenshot_20191002_135647.png</a><br />
Widget preview glows on hover like they used to<br />
<a href="https://phabricator.kde.org/F7496759" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7496759: Screenshot_20191002_135707.png</a></p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Applying style works live</li>
<li class="remarkup-list-item">Applying icon and toolbar settings work live</li>
<li class="remarkup-list-item">Configuring a style works and updates the preview live once you close the style config dialog</li>
</ul>

<p>Two issues remaining</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Just found I broke the kcminit stuff: "Module "kcm_style" was not found or does not actually have a kcminit function"</li>
<li class="remarkup-list-item">On destruction of the qtcurve style it seems to freeze/deadlock, when you close the kcm by clicking "OK" (can't reproduce the backtrace I had but the process doesn't quit)</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/D24361">https://phabricator.kde.org/D24361</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/style/CMakeLists.txt<br />
kcms/style/Messages.sh<br />
kcms/style/kcmstyle.cpp<br />
kcms/style/kcmstyle.h<br />
kcms/style/package/contents/ui/EffectSettingsPopup.qml<br />
kcms/style/package/contents/ui/main.qml<br />
kcms/style/package/metadata.desktop<br />
kcms/style/previewitem.cpp<br />
kcms/style/previewitem.h<br />
kcms/style/styleconfig.ui<br />
kcms/style/stylesmodel.cpp<br />
kcms/style/stylesmodel.h</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>