[Konsole-devel] [Bug 199738] A slight reduction in konsole default line spacing

[John Stanley]

https://bugs.kde.org/show_bug.cgi?id=199738





--- Comment #5 from John Stanley <jpsinthemix verizon net>  2009-07-12 05:25:35 ---
(In reply to comment #2)
> Hi John,
> 
> Can you check what _fm.leading() returns with your font?  Looking at the
> documentation _fm.lineSpacing() returns _fm.height()+_fm.leading() so it must
> be negative in your case.
> 
> Either way, since _lineSpacing is 0, separating lines by
> QFontMetrics::lineSpacing() should give results more like other Qt programs
> rendering the same font.

Hi Robert,
Yup, fm.leading() is -2 for the Monospace and DejaVu Sans Mono fonts. This is
what I get (kde-4.2.96):

font().toString(): DejaVu Sans Mono,7,-1,5,50,0,0,0,0,0
fm.leading():     -2
fm.height():       13
_lineSpacing:      0
fm.lineSpacing():  11

I also attached a .png screenshot: the upper right shell is with the patch I
initially sent, the lower right shell is with it, and the left shell shows the
fm.leading/etc. values for the upper right shell.

An unrelated note: Notice the message in the left shell:
   "Undecodable sequence: \001b(hex)[?1034h

This is the first I've seen of this one... Any thought on this?

(In reply to comment #2)
> Hi John,
> 
> Can you check what _fm.leading() returns with your font?  Looking at the
> documentation _fm.lineSpacing() returns _fm.height()+_fm.leading() so it must
> be negative in your case.
> 
> Either way, since _lineSpacing is 0, separating lines by
> QFontMetrics::lineSpacing() should give results more like other Qt programs
> rendering the same font.

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.