<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D26261">View Revision</a></tr></table><br /><div><div><p>When I apply this and go to the rewritten KCM, the comboboxes are both empty: <a href="https://phabricator.kde.org/F7867971" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7867971: Screenshot_20200107_095737.png</a></p>

<p>Also clicking on the Preview buttons does nothing.</p>

<p>Downloading a new theme using GHNS does not make it appear in the theme list combobox.</p>

<p>On top of that, applying themes doesn't work: <a href="https://phabricator.kde.org/F7867976" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7867976: demo-2020-01-07_09.58.47.webm</a></p>

<p>Given the number of regressions found with simple testing, I'm not comfortable doing this so close to the release of an LTS plasma version. Just in general, huge patches of this size are not really reviewable and don't make sense unless you're also planning on becoming the maintainer.</p>

<p>Given the proximity of the Plasma 5.18 feature freeze, I would suggest finding a way to unify the KCMs that does not require such drastic code changes.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R99 KDE Gtk Configuration Tool</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26261">https://phabricator.kde.org/D26261</a></div></div><br /><div><strong>To: </strong>gikari, apol, VDG, ngraham<br /><strong>Cc: </strong>baberts, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>