For some reason, adding a LTR/RTL Override unicode character before a
MinionPro string increases the ascent of the line.
This creates vertical offset issues when painting if the ascent used
as reference is the one of the font. Use the QTextLine ascent instead
for better results.
This is a followup to
1bed76e2a.
Fixes (reopened) ticket #11284.
Fixes ticket #10532.
Fixes ticket #10767.
(cherry picked from commit
d8498d4785282a6f42a69f56ebd46ac32e1d2f51)
// don't use the pixmap cache
setQPainterPen(computeColor(f.realColor()));
if (dir != Auto) {
- shared_ptr<QTextLayout const> ptl =
- fm.getTextLayout(s, dir == RtL, wordspacing);
- ptl->draw(this, QPointF(x, y - fm.maxAscent()));
- }
- else {
+ auto ptl = fm.getTextLayout(s, dir == RtL, wordspacing);
+ QTextLine const & tline = ptl->lineForTextPosition(0);
+ ptl->draw(this, QPointF(x, y - tline.ascent()));
+ } else {
if (font() != ff)
setFont(ff);
drawText(x, y, str);
- Fix document updating issues with split view (bug 11354).
+- Fix cases where text is offset vertically (bug 10532, 10767).
+
* INTERNALS