D26842: Fix fonts KCM button state

[Fabian Vogt]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200204/d228a40f/attachment.html>