<table><tr><td style="">ahmadsamir created this revision.<br />ahmadsamir added reviewers: Plasma, bport.<br />Herald added a project: Plasma.<br />ahmadsamir requested review of this revision.
</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/D29155">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>It seems that nearestExistingFont() gets the same result of<br />
<tt style="background: #ebebeb; font-size: 13px;">fc-match font-name</tt>, which is basically the sans serif default font<br />
on the system; this seems redundant as KFontChooser will fallback to<br />
selecting the first font family in the list if the initial font it<br />
was called with doesn't exist _and_ KFontChooser always puts "Sans Serif",<br />
"Serif" and "Monospace" at the top of the list.</p>

<p>Removing nearestExistingFont() solves the issue in bug 420287 as<br />
the font it returns will have the styleName property set, after we<br />
went to so much trouble to clear that property so that setBold() can<br />
work.</p>

<p>CCBUG: 420287</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Before the patch:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">In kdeglobals [General] set: fixed=Blah Mono,12,-1,5,50,0,0,0,0,0 toolBarFont=Bogus Serif,12,-1,5,50,0,0,0,0,0</li>
<li class="remarkup-list-item">Load the fonts KCM, and open the font dialog for Fixed and Toolbar, in both cases the default "sans serif" font on the system is selected, in my case it's "DejaVu Sans"</li>
</ul>

<p>Apply the patch and repeat, the "Sans Serif" entry is selected, which is<br />
an alias to "DejaVu Sans" on my system.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>l-setnearest (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29155">https://phabricator.kde.org/D29155</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/fonts/fonts.cpp</div></div></div><br /><div><strong>To: </strong>ahmadsamir, Plasma, bport<br /><strong>Cc: </strong>plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>