Commit 854a62e5 authored by Giusy Margarita's avatar Giusy Margarita Committed by Nate Graham

Remove stepsize 24 from DPI spinbox inside kcm fontconfig

Summary:
Since the spinbox read and write the DPI value "as is", without performing any format change, there's no need to set a stepsize of 24.

As reported [[ https://wiki.archlinux.org/index.php/Font_configuration#Distorted_fonts | here ]], xdpyinfo, xrdb and fontconfig does not expect values multiple of 24 and sometimes the proper DPI value is an unsigned integer different than the default 96, to be manually calculated because xorg can't auto determine it correctly (see [[ https://wiki.archlinux.org/index.php/Xorg#Display_size_and_DPI | here ]]).

Reviewers: #plasma, #vdg, ngraham

Reviewed By: #vdg, ngraham

Subscribers: davidedmundson, ngraham, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D29431
parent 847d9216
......@@ -292,15 +292,12 @@ KCM.SimpleKCM {
QtControls.SpinBox {
id: dpiSpinBox
stepSize: 24
editable: true
enabled: dpiCheckBox.checked
value: kcm.fontsAASettings.dpi !== 0 ? kcm.fontsAASettings.dpi : 96
onValueModified: kcm.fontsAASettings.dpi = value
// to: need to divide to stepSize
to: 1008
// lowest vaue here can be == stepSize, that is because 0 means off
from: 24
to: 999
from: 1
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment