<table><tr><td style="">gikari created this revision.<br />gikari added reviewers: apol, cblack, broulik.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />gikari 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/D25945">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Previously preference of dark GTK3 theme was set by checkbox in GTK KCM. Now, the checkbox is removed and preference is set according to color scheme, the user set in Colors KCMm, using the same heuristic, used for filter combobox in that KCM.</p>
<p>This makes sense, because, suppose the user set Adwaita GTK3 theme, and their color scheme is something dark. Therefore their overall theme is supposedly dark, so Adwaita needs to be dark too.</p>
<p>Note, however, that this preference is somewhat buggy. It does not work in flatpak apps, it can't be autoreloaded with xsettingsd, and also it is applied after some time has passed (this is hypothetical). You can check these statements, using the present checkbox in GTK KCM.</p>
<p>Depends on <a href="https://phabricator.kde.org/D25872" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D25872</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">Apply Adwaita GTK3 Theme</li>
<li class="remarkup-list-item">Reload kded5</li>
<li class="remarkup-list-item">Apply dark color scheme</li>
<li class="remarkup-list-item">Wait some time</li>
<li class="remarkup-list-item">(Re)open GTK3 app</li>
<li class="remarkup-list-item">App should use Dark color scheme</li>
<li class="remarkup-list-item">Apply light color scheme</li>
<li class="remarkup-list-item">Wait some time</li>
<li class="remarkup-list-item">Reopen GTK3 app</li>
<li class="remarkup-list-item">App should use Light color scheme</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R99 KDE Gtk Configuration Tool</div></div></div><br /><div><strong>BRANCH</strong><div><div>sync-dark-gtk3-theme-preference (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25945">https://phabricator.kde.org/D25945</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
kded/configvalueprovider.cpp<br />
kded/configvalueprovider.h<br />
kded/gtkconfig.cpp<br />
kded/gtkconfig.h<br />
src/appearancegtk3.cpp<br />
src/appearancegtk3.h<br />
src/appearencegtk.cpp<br />
src/appearencegtk.h<br />
src/gtkconfigkcmodule.cpp<br />
src/ui/gui.ui<br />
tests/configsavetest.cpp</div></div></div><br /><div><strong>To: </strong>gikari, apol, cblack, broulik<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, ahiemstra, mart<br /></div>