Bug 467326

Summary:	sddm-kcm should also synchronize CursorSize
Product:	[Applications] systemsettings	Reporter:	Andrew Ammerlaan <andrewammerlaan>
Component:	kcm_sddm	Assignee:	David Edmundson <kde>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	agurenko, geraldo.simiao.kutz, kde, nate, plasma-bugs, shkadinskiy98
Priority:	NOR
Version:	5.27.2
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/sddm-kcm/commit/0a7e6f596d7496c0a3385f55378363ec51377ab2	Version Fixed In:	5.27.6
Attachments:	apply plasma settings - reset to default apply plasma settings - apply

Description Andrew Ammerlaan 2023-03-14 11:44:19 UTC

SUMMARY

I was trying out the native wayland support in SDDM git and found that the cursor looked way to big. The problem was easily fixable by setting 'CursorSize=24' in sddm.conf.d. However, this should probably be synchronized as well when the user presses the "Apply Plasma Settings" button. 

STEPS TO REPRODUCE
1. Upgrade to sddm-git
2. Switch sddm to wayland with 'DisplayServer=wayland'
3. Synchronize settings from plasma

OBSERVED RESULT

Currently synchronizing the Plasma settings to sddm gives me this kde_settings.conf:
[Autologin]
Relogin=false
Session=
User=

[General]
HaltCommand=systemctl shutdown
Numlock=on
RebootCommand=systemctl reboot

[Theme]
Current=breeze
CursorTheme=breeze_cursors
Font=Noto Sans [GOOG],10,-1,5,50,0,0,0,0,0,Regular

[Users]
MaximumUid=65000
MinimumUid=1000

[X11]
ServerArguments=-dpi 0

EXPECTED RESULT

The CursorSize setting is also synchronized automatically, leading to a uniform cursor size when logging in and loading a Plasma session.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Gentoo Linux 6.3.0_rc1
KDE Plasma Version:  5.27.2
KDE Frameworks Version: 5.104.0
Qt Version: 5.15.8

Comment 1 Bug Janitor Service 2023-03-14 20:23:39 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/sddm-kcm/-/merge_requests/41

Comment 2 Nate Graham 2023-03-15 13:19:15 UTC

Git commit 650098c27f1a8c6b2ccba448c2c0aead57ca9a45 by Nate Graham.
Committed on 14/03/2023 at 20:23.
Pushed by ngraham into branch 'master'.

Sync cursor theme too

SDDM supports this; let's sync it.
FIXED-IN: 6.0

M  +1    -1    src/package/contents/ui/main.qml
M  +6    -0    src/sddmkcm.cpp

https://invent.kde.org/plasma/sddm-kcm/commit/650098c27f1a8c6b2ccba448c2c0aead57ca9a45

Comment 3 Nate Graham 2023-03-25 20:58:10 UTC

*** Bug 436015 has been marked as a duplicate of this bug. ***

Comment 4 Nate Graham 2023-04-16 16:23:58 UTC

Git commit b55a56b1fd6e97095c72dfe4d764a57c147a4d5a by Nate Graham.
Committed on 16/04/2023 at 16:22.
Pushed by ngraham into branch 'master'.

Reset cursor size along with everything else when reverting to defaults

Forgotten in 650098c27f1a8c6b2ccba448c2c0aead57ca9a45.

M  +2    -0    src/sddmkcm.cpp

https://invent.kde.org/plasma/sddm-kcm/commit/b55a56b1fd6e97095c72dfe4d764a57c147a4d5a

Comment 5 Nate Graham 2023-04-20 23:47:09 UTC

Git commit 0a7e6f596d7496c0a3385f55378363ec51377ab2 by Nate Graham.
Committed on 20/04/2023 at 23:45.
Pushed by ngraham into branch 'Plasma/5.27'.

Sync cursor size too

SDDM supports this; let's sync it.
FIXED-IN: 5.27.5

(cherry picked from commit 650098c27f1a8c6b2ccba448c2c0aead57ca9a45)
(cherry picked from commit b55a56b1fd6e97095c72dfe4d764a57c147a4d5a)

M  +8    -0    src/sddmkcm.cpp

https://invent.kde.org/plasma/sddm-kcm/commit/0a7e6f596d7496c0a3385f55378363ec51377ab2

Comment 6 Nate Graham 2023-05-22 19:37:55 UTC

*** Bug 470024 has been marked as a duplicate of this bug. ***

Comment 7 Geraldo Simião 2023-05-26 11:51:59 UTC

I tested on this setup and bug still present:    

    Operating System: Fedora Linux 38
    KDE Plasma Version: 5.27.5
    KDE Frameworks Version: 5.106.0
    Qt Version: 5.15.9
    Kernel Version: 6.3.3-200.fc38.x86_64 (64-bit)
    Graphics Platform: Wayland

    sddm-0.19.0^git20230404.e652433-1.fc38.x86_64
    kde-settings-sddm-38.2-2.fc38.noarch
    sddm-wayland-plasma-5.27.5-1.fc38.noarch
    sddm-breeze-5.27.5-1.fc38.noarch
    sddm-kcm-5.27.5-1.fc38.x86_64

    plasma-systemsettings-5.27.5-1.fc38.x86_64

Comment 8 Geraldo Simião 2023-05-26 12:28:49 UTC

Created attachment 159267 [details]
apply plasma settings - reset to default

sddm.conf.d/kde_settings.conf after reset to default on system settings

Comment 9 Geraldo Simião 2023-05-26 12:30:26 UTC

Created attachment 159268 [details]
apply plasma settings - apply

sddm.conf.d/kde_settings.conf after applying current settings (correct cursor size was 24 on system settings)

Comment 10 Nate Graham 2023-05-26 13:12:18 UTC

Hmm, these reports are strange; I tested this and it was working for me when I merged it. I'll investigate.