<table><tr><td style="">rkflx created this revision.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D6697" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The fonts KCM supports changing the toolbar font, but with the port to<br />
QPA/Plasma 5 in <a href="https://phabricator.kde.org/R446:aa16577f402549c73721164aa45141caff774142" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">aa16577f</a> this stopped working and toolbars just used the<br />
general font. This is fixed by setting QPlatformTheme's ToolButtonFont<br />
to the toolbar font chosen. However, there are some caveats:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Default toolbar font size is 9, while the general font size is 10.</li>
</ul>

<p>This would result in a sudden shrink in toolbar widths everywhere (or<br />
worse for weird leftover settings users might have tried unsuccessfully).</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">In some places QToolButtons are used outside of toolbars, sometimes</li>
</ul>

<p>right next to QPushButtons. As due to the bug both had the same font,<br />
this might now lead to unexpected differences.</p>

<p>Therefore, to keep visual consistency with past Plasma 5 releases, let's<br />
change the default toolbar font size to 10 and provide a kconf_update<br />
script to copy the general font to the toolbar font once. This should<br />
give us some time to fix fonts of single QToolButtons and discuss with<br />
the <a href="https://phabricator.kde.org/tag/vdg/" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">#VDG</a> whether to leave the default toolbar font size at 10 or change<br />
it back to 9 with a major release as originally intended.</p>

<p>BUG: 358254</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">sudo make install, verify *.{so,upd,pl} are in the same location as</li>
</ul>

<p>the regularly installed files: toolBarFont in kdeglobals gets updated<br />
immediately</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Change "Toolbar" font in "kcmshell5 fonts", start your favorite KDE</li>
</ul>

<p>and Qt-only applications: toolbar font and/or font size changed<br />
accordingly</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R135 Integration for Qt applications in Plasma</div></div></div><br /><div><strong>BRANCH</strong><div><div>Plasma/5.10</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6697" rel="noreferrer">https://phabricator.kde.org/D6697</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/platformtheme/CMakeLists.txt<br />
src/platformtheme/fonts_global_toolbar.pl<br />
src/platformtheme/fonts_global_toolbar.upd<br />
src/platformtheme/kdeplatformtheme.cpp<br />
src/platformtheme/kfontsettingsdata.cpp</div></div></div><br /><div><strong>To: </strong>rkflx, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>