dfaust / plasma-applet-netspeed-widget

Plasma 5 widget that displays the currently used network bandwidth
GNU General Public License v2.0
128 stars 17 forks source link

Poorly rendered fonts (with fix) #17

Closed Focshole closed 5 years ago

Focshole commented 5 years ago

Good Evening, As first I'd like to thank you for this awesome work, it works very well. The only issue I have found is on font rendering. As you can see here: Image_poorly_rendered I have fixed it by adding the property renderType: Text.NativeRendering to Text units with id TopText, topUnitText, bottomText,bottomUnitText in file CompactRepresentation.qml. The result is: Image_well_rendered Which is much more clear to read.

dfaust commented 5 years ago

Thank you. I hope this won't break anything. It doesn't make a difference on my system.

dfaust commented 5 years ago

@Focshole Do you have this problem with other QML widgets as well?

Focshole commented 5 years ago

This was the worst rendering one. From the manual https://doc-snapshots.qt.io/qt5-5.9/qml-qtquick-text.html#renderType-prop it says:

Select Text.NativeRendering if you prefer text to look native on the target platform and do not require advanced features such as transformation of the text. Using such features in combination with the NativeRendering render type will lend poor and sometimes pixelated results.

After noticing that no text transformations have been applied in the plasmoid code, i thought it could work. It was poorly rendering also on a laptop of a friend of mine.