<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/D22844">View Revision</a></tr></table><br /><div><div><p>Thanks, it works much better now. I'm really impressed with this, and it's very cool to be able to see GTK3 apps respecting the system color scheme:<br />
<a href="https://phabricator.kde.org/F7143222" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7143222: Screenshot_20190802_141140.png</a></p>

<p>However this is still not shippable in its current state due to a reproducible issue: GTK apps will only reliable change their color scheme after you apply a new color scheme in the KCM <em>twice</em>. If you only apply it once, most or all of the time, GTK apps will not change, and will retain the old color scheme. This needs to be fixed first so that applying the new color scheme once is all it takes.</p>

<p>Also, it's a bit annoying that you need to restart GTK apps to see the change, but I suppose that's inevitable (and if fixable, we can fix it in a subsequent patch).</p></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/D22844">https://phabricator.kde.org/D22844</a></div></div><br /><div><strong>To: </strong>cblack, Plasma, ngraham<br /><strong>Cc: </strong>ngraham, davidre, broulik, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>