D12925: Parse global config files. Remove 'Vendor default' option. Fix changes not recognized.
Julian Wolff
noreply at phabricator.kde.org
Wed May 16 09:16:41 UTC 2018
progwolff created this revision.
progwolff added a reviewer: rkflx.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
progwolff requested review of this revision.
REVISION SUMMARY
The fonts kcm did only parse a local config file. Global and default settings were not recognized.
With this patch, all global config files are parsed before the local file is parsed.
This allows us to drop the "Vendor default" option.
This patch also fixes some changes not recognized (apply button disabled).
TEST PLAN
Delete ~/.config/fontconfig/fonts.conf.
Start `kcmshell5 fonts`. The default/global settings should be displayed.
Change some settings. The apply button should be enabled.
Save. A new config file should be created.
REPOSITORY
R119 Plasma Desktop
BRANCH
fonts_kcm_globalConfigFiles (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D12925
AFFECTED FILES
kcms/fonts/fonts.cpp
kcms/fonts/fonts.h
kcms/fonts/kxftconfig.cpp
kcms/fonts/kxftconfig.h
kcms/fonts/package/contents/ui/main.qml
To: progwolff, rkflx
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180516/a38bde7d/attachment-0001.html>
More information about the Plasma-devel
mailing list