<table><tr><td style="">ngraham accepted this revision.<br />ngraham added a comment.<br />This revision is now accepted and ready to land.
</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/D29064">View Revision</a></tr></table><br /><div><div><p>Works great, and I fully support the overall goal of getting rid of kcmshell. Doing so will fix a ton of bugs (all the systemsettings | kcmshell bugs as well as all the bug reports about weird sizing in standalone KCMs, plus a bunch more about inconsistent behavior of KCMs in systemsettings vs kcmshell)</p>

<p>Foor fod thought: we might want to also think about some of the consequences of removing kcmshell; we could need to add programmatic cross-KCM navigation into system settings itself. For example, in <a href="https://phabricator.kde.org/D29080" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D29080</a>, I'm calling kcmshell from inside a KCM to pop-up the KScreen KCM on top of the fonts KCM. Without kcmshell, that won't work as currently implemented, and it would need to navigate you to the KScreen KCM in the same systemsettings instance.</p>

<p>Also a new way to get the information from <tt style="background: #ebebeb; font-size: 13px;">kcmshell5 --list</tt> would be nice.</p>

<p>Material for follow-up patches, I suppose.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R124 System Settings</div></div></div><br /><div><strong>BRANCH</strong><div><div>phab/startupModule</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29064">https://phabricator.kde.org/D29064</a></div></div><br /><div><strong>To: </strong>mart, Plasma, ngraham<br /><strong>Cc: </strong>bport, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>