When saving, kcm_fonts adds the default selected StyleName,
so it needs to specified here as well to avoid mismatches.
BUG: 383191
davidedmundson |
Plasma |
When saving, kcm_fonts adds the default selected StyleName,
so it needs to specified here as well to avoid mismatches.
BUG: 383191
Ran pkcon install with no kdeglobals.
Without this change: Bold title in the auth dialog
With this change: Normal title in the auth dialog
No Linters Available |
No Unit Test Coverage |
Note that saving the style name is a bug, see https://bugs.kde.org/show_bug.cgi?id=378523
While I'm not 100% sure that is true as that's how the user selects the font in kcm_fonts, I still think this patch is
by itself an improvement as the stylenames are provided by the font creators to ship predefined metrics for aesthetics.
Possibly related: https://github.com/qt/qtbase/commit/2bb01172a76f14b1f57010b79207f6b9a9bf3f53
So still ok to land?
Didn't get closed automatically again...
https://commits.kde.org/plasma-integration/1b21b5977c2068c5bd30c9f9f641f60bdba9ea8e