<table><tr><td style="">cfeck added a reviewer: cfeck.<br />cfeck added a comment.
</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/D9070" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The config would work with removed styleName() for the cases where the weight() and style() attributes are sufficient. The intention from Qt's side was to uniquely identify fonts that cannot be selected by those two attributes. Imagine a font that has these faces:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">HandwriteRegular</li>
<li class="remarkup-list-item">HandwriteBold</li>
<li class="remarkup-list-item">HandwriteItalic</li>
<li class="remarkup-list-item">HandwriteBoldItalic</li>
<li class="remarkup-list-item">HandwriteCursive</li>
<li class="remarkup-list-item">HandwriteBoldCursive</li>
</ul>

<p>Now the user picks "HandwriteCursive", and unless we do not save the styleName(), the font matching would probably select "HandwriteItalic" based on the weight() and style() attributes. On the other hand, calling setBold(true) on a font called "HandwriteCursive" _should_ select "HandwriteBoldCursive", even if the styleName was intended to enforce a specific face. If Qt cannot fix this issue, then we have to clear the styleName() at least for those fonts, where a perfect match is possible using the attributes alone (by comparing with the database again).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R135 Integration for Qt applications in Plasma</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9070" rel="noreferrer">https://phabricator.kde.org/D9070</a></div></div><br /><div><strong>To: </strong>rjvbb, Frameworks, davidedmundson, graesslin, cfeck<br /><strong>Cc: </strong>cfeck, fvogt, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>