hetzenecker added a comment.
Awesome! Looks good, this fixes the wrong icon sizes in plasma.
"xrdb -query" reports the correct size and .config/kcmfonts now also stores
the integer value. This means the "Force fonts DPI" size is now also shown in
the Fonts systemsettings module.
Tes
hetzenecker added inline comments.
INLINE COMMENTS
> scalingconfig.cpp:124
> {
> -return scaleFactor() * 96.0;
> +return qRound(scaleFactor()) * 96.0;
> }
I'd propse
qRound(scaleFactor() * 96.0)
to get int values. If the scaleFactor is rounded, we only get multiplies of 96
as DPI.
