https://bugs.kde.org/show_bug.cgi?id=461483
ratijas <m...@ratijas.tk> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://invent.kde.org/plas
| |ma/plasma-nm/-/commit/275ea
| |cf2088e84d5d4d70531a976ea3d
| |807b0ebd
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #6 from ratijas <m...@ratijas.tk> ---
Git commit 275eacf2088e84d5d4d70531a976ea3d807b0ebd by ivan tkachenko.
Committed on 01/12/2023 at 17:57.
Pushed by ratijas into branch 'master'.
applet: Fix layout of the network speed graph with certain font preferences
TextMetrics uses a different text rendering backend by default, which
has led to inaccurate estimation of the width of AxisLabels. By default
TextMetrics uses QtRendering type; but our Label uses NativeRendering.
In past we used to override renderType with a conditional expression,
so considering such history, hardcoding any type seems inappropriate.
That's why we need an actual instance of Label to fetch a correct
render type at runtime.
Second part of the issue was that instead of width property the layout
should have used advanceWidth, and ceil() it, just in case. But with a
port to Label instance it doesn't matter anymore.
M +9 -6 applet/contents/ui/TrafficMonitor.qml
https://invent.kde.org/plasma/plasma-nm/-/commit/275eacf2088e84d5d4d70531a976ea3d807b0ebd
--
You are receiving this mail because:
You are watching all bug changes.
