This patch adds scaling to the list of Plasma settings that can be synced with SDDM.
- Group Reviewers
- R123:f909a8ab0ea3: Sync scaling DPI from Plasma to SDDM
Displays->Scale Display->set to 1.5
Login Screen(SDDM)->Advanced->pressed "Sync"
SDDM was big.
One potentially major issue that if I'm right needs fixing ASAP
What are you going to write if this entry is missing?
I suspect you'll write just
"-dpi " into ServerArguments
and then X11 will run with "X11 -dpi" expect another argument and simply fail to load?
As a coding note it's useful to make sure the data in the variables matches their name.
In this case
"dpiValue" isn't just the DPI value, it's a string formatted as X server arguments "-dpi" here.
So I would suggest either renaming or (probably easier with the other fix) adding the "-dpi "prefix when you insert it into the map.
I believe you're right. Let me know if this is a good fix: https://cgit.kde.org/sddm-kcm.git/commit/?id=a48e07c8156a10a7519f18ec72a9608fc5bfaa0a