<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/D18247">View Revision</a></tr></table><br /><div><div><p>+1. The only change here that I think might be  remotely controversial is changing "Desktop theme" to Plasma theme". But I thin that's a very good change since it improves the ability to understand what this actually is, and also uses our branding. You're using Plasma, so this thing chooses the Plasma theme. Makes sense to me.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>improve-desktoptheme-kcm (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18247">https://phabricator.kde.org/D18247</a></div></div><br /><div><strong>To: </strong>GB_2, Plasma, VDG, ngraham<br /><strong>Cc: </strong>ngraham, davidedmundson, VDG, plasma-devel, Plasma, sukalyanbanga, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>