D9070: KDE platform plugin: don't force default stylename on user-specified fonts

[René J.V. Bertin]

rjvbb added a comment.


  >   So this this actually resolve https://bugs.kde.org/show_bug.cgi?id=378523, or just lay the groundwork for resolving it?
  
  No, it's more a change similar to proposed fix for that bug (don't set a stylename in KFontRequester).
  This patch implements the principle that you shouldn't set a style name if you cannot guarantee that the side-effects of that change are without consequence. The platform theme plugin loads the desktop fonts and if it imposes a style name on them no application can set them to bold or italic anymore.
  
  So you could say that this patch is necessary for the KFontRequester fix to work as intended (as far as the desktop fonts are concerned).

REPOSITORY
  R135 Integration for Qt applications in Plasma

REVISION DETAIL
  https://phabricator.kde.org/D9070

To: rjvbb, #frameworks, davidedmundson, graesslin, cfeck, dfaure
Cc: anthonyfieroni, ngraham, cfeck, fvogt, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171230/638867b1/attachment.html>