D7160: Also specify a default StyleName for fonts

2017-08-11 Thread Fabian Vogt
fvogt closed this revision. fvogt added a comment. Didn't get closed automatically again... https://commits.kde.org/plasma-integration/1b21b5977c2068c5bd30c9f9f641f60bdba9ea8e REPOSITORY R135 Integration for Qt applications in Plasma REVISION DETAIL

D7160: Also specify a default StyleName for fonts

2017-08-06 Thread Fabian Vogt
fvogt added a comment. In https://phabricator.kde.org/D7160#132870, @cfeck wrote: > 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

D7160: Also specify a default StyleName for fonts

2017-08-06 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R135 Integration for Qt applications in Plasma BRANCH stylename REVISION DETAIL https://phabricator.kde.org/D7160 To: fvogt, #plasma, davidedmundson Cc: plasma-devel, ZrenBot, progwolff,

D7160: Also specify a default StyleName for fonts

2017-08-06 Thread Christoph Feck
cfeck added a comment. Note that saving the style name is a bug, see https://bugs.kde.org/show_bug.cgi?id=378523 REPOSITORY R135 Integration for Qt applications in Plasma BRANCH stylename REVISION DETAIL https://phabricator.kde.org/D7160 To: fvogt, #plasma, davidedmundson Cc: cfeck,

D7160: Also specify a default StyleName for fonts

2017-08-06 Thread Fabian Vogt
fvogt created this revision. Restricted Application added a project: Plasma. REVISION SUMMARY When saving, kcm_fonts adds the default selected StyleName, so it needs to specified here as well to avoid mismatches. BUG: 383191 TEST PLAN Ran pkcon install with no kdeglobals. Without