<table><tr><td style="">dhaumann 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/D6695" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>With my Kate hat on I have to add that some fonts are quite broken, i.e. it may very well be that QFontMetrics has no chance in ever always returning accurate metrics (we have to fight issues like that in katepart in quite ugly ways). That said, this patch fixes a problem now by tweaking the paddling left and right. Since this is about 10 pixels in total, I would say this workaround is better than the graphical artifacts we otherwise get.</p>

<p>A proper fix in Qt (or maybe even in the font or similar) is certainly better, but unlikely to happen soon. With this in mind, the patch looks good to me.</p>

<p>In any case, someone else should decide whether this solution is ok, or whether there is a better fix in a realistic timeframe.</p>

<p><a href="https://phabricator.kde.org/p/rkflx/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@rkflx</a> What you could try is to make a small test that shows this issue. Maybe this only appears with certain fonts at certain font sizes. In case you can reproduce, you should open a Qt bug report. Maybe one already exists...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6695" rel="noreferrer">https://phabricator.kde.org/D6695</a></div></div><br /><div><strong>To: </strong>rkflx, Okular<br /><strong>Cc: </strong>dhaumann, aacid<br /></div>