Bug 372218 - Task switcher "Compact" ignores font anti-aliasing settings
Summary: Task switcher "Compact" ignores font anti-aliasing settings
Status: RESOLVED FIXED
Alias: None
Product: kdeplasma-addons
Classification: Plasma
Component: General (show other bugs)
Version: unspecified
Platform: Kubuntu Linux
: NOR minor
Target Milestone: ---
Assignee: Kai Uwe Broulik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-08 15:45 UTC by Daniel T.
Modified: 2016-11-14 09:26 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.8.4
Sentry Crash Report:


Attachments
Task switcher "Compact" (29.46 KB, image/png)
2016-11-08 15:45 UTC, Daniel T.
Details
Compare fonts in Task Switcher "Compact" and elsewhere (454.69 KB, image/png)
2016-11-08 15:51 UTC, Daniel T.
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel T. 2016-11-08 15:45:12 UTC
Created attachment 102124 [details]
Task switcher "Compact"

I enabled anti-aliasing in Fonts System Setting and configured it as following

sub-pixel rendering type: RGB 
hinting style: full

However, this setting does not seem to have an effect on the Task Switcher (Alt+Tab) which is set to Compact. The fonts in it are not anti-aliased properly.
Comment 1 Daniel T. 2016-11-08 15:51:28 UTC
Created attachment 102125 [details]
Compare fonts in Task Switcher "Compact" and elsewhere

The fonts in the Task Switcher "Compact" are blurred (not properly anti-aliased), while the fonts elsewhere are crisp.
Comment 2 Christoph Feck 2016-11-09 00:31:23 UTC
QtQuick allows native freetype text rendering (which is slower than the default OpenGL distance field rendering), but it has to be enabled using a text property, see http://doc.qt.io/qt-5/qml-qtquick-text.html#renderType-prop
Comment 3 Martin Flöser 2016-11-09 07:38:15 UTC
Moving to kdeplasma-addons as the compact switcher is part of addons.
Comment 4 Daniel T. 2016-11-09 13:49:17 UTC
Is there a way to try that out? Where should I add this text property?
Comment 5 Kai Uwe Broulik 2016-11-09 18:00:41 UTC
Just looked at the source code - it is just a plain Text item, not PlasmaComponents Label - so it doesn't use native rendering. I'll have a look.
Comment 6 Kai Uwe Broulik 2016-11-10 14:25:23 UTC
Please give the following patch a try: https://phabricator.kde.org/D3330
Comment 7 Kai Uwe Broulik 2016-11-14 09:26:06 UTC
Git commit 00369ec41315470ebf121bd12d8309ddf9ed1b87 by Kai Uwe Broulik.
Committed on 14/11/2016 at 09:25.
Pushed by broulik into branch 'Plasma/5.8'.

[Window/Desktop Switchers] Use PlasmaComponents.Label instead of Text

The former uses native text rendering which respects anti-aliasing and hinting settings.
It also automatically follows theme colors.
Unfortunately it breaks the font.bold binding, so using font.weight here instead.
FIXED-IN: 5.8.4

CHANGELOG: Font rendering in window/desktop switchers now follows anti-aliasing and hinting settings

Differential Revision: https://phabricator.kde.org/D3330

M  +3    -5    desktopswitchers/previews/contents/ui/main.qml
M  +3    -5    windowswitchers/big_icons/contents/ui/main.qml
M  +3    -3    windowswitchers/compact/contents/ui/main.qml
M  +4    -6    windowswitchers/informative/contents/ui/main.qml
M  +3    -3    windowswitchers/present_windows/contents/ui/main.qml
M  +3    -5    windowswitchers/small_icons/contents/ui/main.qml
M  +2    -2    windowswitchers/text/contents/ui/main.qml
M  +3    -5    windowswitchers/thumbnails/contents/ui/main.qml

http://commits.kde.org/kdeplasma-addons/00369ec41315470ebf121bd12d8309ddf9ed1b87