Round label width
ClosedPublic

Authored by fvogt on Wed, Dec 5, 9:35 AM.

Details

Summary

Currently the icon labels can be at non-integer positions.
This results in ugly font rendering.

Test Plan

Before/after:

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
fvogt created this revision.Wed, Dec 5, 9:35 AM
Restricted Application added a project: Plasma. · View Herald TranscriptWed, Dec 5, 9:35 AM
fvogt requested review of this revision.Wed, Dec 5, 9:35 AM
fvogt edited the test plan for this revision. (Show Details)Wed, Dec 5, 9:46 AM
broulik added a subscriber: broulik.Wed, Dec 5, 9:48 AM
broulik added inline comments.
containments/desktop/package/contents/ui/FolderItemDelegate.qml
66

That's what this hideous code is actually supposed to do

fvogt added inline comments.Wed, Dec 5, 9:50 AM
containments/desktop/package/contents/ui/FolderItemDelegate.qml
66

It doesn't work though - I'll check some more.

fvogt updated this revision to Diff 46884.Wed, Dec 5, 9:51 AM

Turns out the label width rounding is enough

fvogt retitled this revision from Round folder view delegate and label positions to Round label width.Wed, Dec 5, 9:52 AM
fvogt edited the summary of this revision. (Show Details)
fvogt marked 2 inline comments as done.
broulik accepted this revision.Wed, Dec 5, 9:53 AM

Cool! Stable branch pls

This revision is now accepted and ready to land.Wed, Dec 5, 9:53 AM
This revision was automatically updated to reflect the committed changes.
ngraham added a subscriber: ngraham.Wed, Dec 5, 2:44 PM

Hmm, the "After" pictured depicts terrible kerning. Take a look at "Discover" in particular. Is this really the trade-off we're faced with? It feels like there are upstream Qt bugs here.

fvogt added a comment.Wed, Dec 5, 3:14 PM

Hmm, the "After" pictured depicts terrible kerning. Take a look at "Discover" in particular. Is this really the trade-off we're faced with? It feels like there are upstream Qt bugs here.

Yup, definitely. Font rendering in QML has some issues.

FreeType hinting configuration and especially the used font can make a huge difference though. With "Roboto" for example it looks perfectly fine.