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

Reply via email to