broulik created this revision. broulik added reviewers: Plasma, davidedmundson. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision.
REVISION SUMMARY We copy the `.colors` file's contents into `kdeglobals` and I have seen many of them containing bogus [General] ColorScheme=Some Other Scheme which makes no sense. Ensure the `ColorScheme` that ends up in `kdeglobals` is the scheme defined by the look and feel package. BUG: 403774 TEST PLAN Downloaded "Modern" lnf from kde store, applied it and system changed to a dark color scheme. Previously it would write "Breeze Dark" from its colors file to kdeglobals, now it is correctly "BreezeDarcBlue" This also obsoletes D18752 <https://phabricator.kde.org/D18752> as now the correct scheme name is always present in kdeglobals REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D18754 AFFECTED FILES kcms/lookandfeel/kcm.cpp To: broulik, #plasma, davidedmundson Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart