https://bugs.kde.org/show_bug.cgi?id=386566

--- Comment #8 from Bhushan Shah <bhus...@gmail.com> ---
Git commit 39c6facc51a807acc418a2e0dad6fda37d53071f by Bhushan Shah, on behalf
of Julian Wolff.
Committed on 27/05/2019 at 09:06.
Pushed by bshah into branch 'master'.

Parse global config files. Remove 'Vendor default' option. Fix changes not
recognized.

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` and `~/.config/kcmfonts` .
Run `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.

Follow up of the D12925

M  +146  -81   kcms/fonts/fonts.cpp
M  +28   -16   kcms/fonts/fonts.h
M  +209  -95   kcms/fonts/kxftconfig.cpp
M  +11   -0    kcms/fonts/kxftconfig.h
M  +13   -19   kcms/fonts/package/contents/ui/main.qml
M  +9    -6    kcms/fonts/previewimageprovider.cpp

https://commits.kde.org/plasma-desktop/39c6facc51a807acc418a2e0dad6fda37d53071f

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to