fvogt closed this revision. fvogt added a comment.
In D26842#605850 <https://phabricator.kde.org/D26842#605850>, @bport wrote: > In D26842#605839 <https://phabricator.kde.org/D26842#605839>, @fvogt wrote: > > > Does this mean the default fonts have a style name of "Regular" now? That will break setting bold fonts and such, see https://phabricator.kde.org/D9070. > > > Default is not serialized in the configuration file (because it's the default value). > > I do some check with current KCM version, and when we save it, we end up with font=Noto Sans,10,-1,5,87,1,0,0,0,0,Regular > Because nearest font algorithm return the regular one. So if I well understood everything this patch will not break anything. Ok, sounds good. Thanks for checking! REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D26842 To: bport, #plasma, ervin, crossi, meven, ngraham Cc: fvogt, broulik, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
