|
Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
| Summary: | Descenders get cropped within OS/X widgets | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | General | Reporter: | Thorsten Wagner <thorsten.wagner.4> | ||||||||
| Component: | ui | Assignee: | AOO issues mailing list <issues> | ||||||||
| Status: | UNCONFIRMED --- | QA Contact: | |||||||||
| Severity: | Normal | ||||||||||
| Priority: | P3 | CC: | hdu | ||||||||
| Version: | 4.2.0-dev | ||||||||||
| Target Milestone: | --- | ||||||||||
| Hardware: | Mac | ||||||||||
| OS: | OS X 10.9 | ||||||||||
| Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||||||
| Developer Difficulty: | --- | ||||||||||
| Attachments: |
|
||||||||||
Created attachment 83487 [details]
Screenshot without patch (AOO 4.1)
Created attachment 83488 [details]
Screenshot with patch (AOO 4.2)
Comment on attachment 83486 [details]
Patch
Added "patch" flag to the attachment to allow easy diff-viewing.
Forcing round-down instead of round-nearest results in worse metrics so I'd rather avoid this. Please check whether the rounding fixes for the repaint region (revision 1599173 and revision 1599174) already help for the displays in question. Unfortunately the fixes in revision 1599173 and revision 1599174 don't solve the problem. The result is still as shown in attached screenshot without patch. Furthermore the attached patch doesn't solve the problem too. I discovered side effects (characters within status bar become croped and height of tool bars is calculated too small now). |
Created attachment 83486 [details] Patch Using Mac OS/X depending on screen resolution different character heights are calculated: (1) The calculation of character heights in "ctfonts.cxx" works fine for laptop screens including retina at different resolutions. (2) For larger cinema displays characters are calculated a little bit too tall. This causes cropping of descenders within widgets (see attached screenshot 1, red boxes). I supplied a patch with modified roundings. The patch causes character heights to be rounded down to the next pixel instead of rounding to the nearest pixel. The result is shown in attached screenshot 2. The patch needs to be doublechecked to avoid side effects.