<table><tr><td style="">mglb 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/D19266">View Revision</a></tr></table><br /><div><div><p>Depends on your opinion. As for me:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">It is nice that we get all possible information about what an user selected in the font dialog - we can remove/change things we don't need/support.</li>
<li class="remarkup-list-item">The style name allows for most precise (afaik) style specification. Technically, there can exist two styles, with the same weight, stretch, etc, but different glyphs. However, Qt should unset styleName after manual change of particular style properties. It doesn't, so we need to do it manually (this can be considered the problem to which the patch is a workaround).</li>
</ul>

<p>If you just mean "does this fix that bug" - yes.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19266">https://phabricator.kde.org/D19266</a></div></div><br /><div><strong>To: </strong>mglb, Konsole, VDG<br /><strong>Cc: </strong>ngraham, konsole-devel, gennad, maciejn, thsurrel, maximilianocuria, hindenburg<br /></div>