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
