Bug 454528 - The "Apply" button doesn't wok properly on Login Screen (SDDM) page
Summary: The "Apply" button doesn't wok properly on Login Screen (SDDM) page
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_sddm (show other bugs)
Version: 5.24.90
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: David Edmundson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-05-28 10:46 UTC by John
Modified: 2022-08-09 23:33 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.26


Attachments
The "user" combobox may not match the saved value. (695.59 KB, video/mp4)
2022-06-20 21:02 UTC, oioi555x
Details

Note You need to log in before you can comment on or make changes to this bug.
Description John 2022-05-28 10:46:21 UTC
SUMMARY
The "Apply" button doesn't wok properly on Login Screen (SDDM) page


STEPS TO REPRODUCE
1. Go to System Setting -> Startup and Shutdown -> Login Screen (SDDM) -> Behavior...
2. Uncheck the "Log in again immediately after logging off"
3. Hit the "Apply" button to save changes

OBSERVED RESULT
The "Apply" button is still clickable and you have to click it again.
If try to close the window after you have clicked it once, it will ask you to apply the changes so clicking once was definitely not enough, but if you choose "Discard" and reopen the same location again you will see that the changes were actually applied as that checkbox is not ticked anymore.
Now, if you reopen the same location and want to do the reverse, to tick that checkbox, you can't as it's disabled
To make it enabled again you have to tick also the checkbox next to (on the right of) the "Automatically log in" and choose a user, but that drop-down box seems to not show any options.

EXPECTED RESULT
After ticking that checkbox hitting the "Apply" button once should be enough with the button showing pressed and not letting you to click it again and also closing the window should not ask you to apply the changes.
Opening the same page to do the reverse, should be just as easy and the checkbox should be tickable without  needing to configure the other stuff as the other stuff was not configured when it was ticked and enabled last time.
If the user unticked a single checkob to disable somehting, then it should be able to tick that single checkbox to enable it again
Also that drop-down with the users should display some values, at least the current user

SOFTWARE/OS VERSIONS
Linux/KDE Plasma:
KDE Plasma Version: 5.24.90
KDE Frameworks Version: 5.95
Qt Version: 5.15.4

ADDITIONAL INFORMATION
From: neon-testing-20220524-1822.iso + updates from Discover
Tested on Wayland
Comment 1 oioi555x 2022-06-05 12:25:35 UTC
I felt the "Apply" button the same as BUG 437423.
However, your problem could not be reproduced.

"Log in again immediately after logging off" without "Automatically log in" does not work even if it can be specified.
So I think it's correct that checkbox is disabled.

The reason why user drop-downs are empty is
The "Minimum user UID" and "Maximum user UID" settings may be incorrect.

Normally you can restore with the "Default" button,
There seems to be a problem with saving, so please try deleting the configuration file directly.

/etc/sddm.conf.d/kde_settings.conf
Comment 2 oioi555x 2022-06-20 20:59:04 UTC
I rechecked the auto-login settings page and found some issues.  

1. The "user" combobox may not match the saved value.  
2. The "user" and "session" comboboxes does not return value with the "reset" button.  

The "Apply" button looks confusing when encountering these issues.
Comment 3 oioi555x 2022-06-20 21:02:26 UTC
Created attachment 149971 [details]
The "user" combobox may not match the saved value.
Comment 4 Nate Graham 2022-08-09 23:33:21 UTC
Git commit d5dcd878dcd004a00885410f7209f169c68cdc23 by Nate Graham, on behalf of oioi 555.
Committed on 09/08/2022 at 23:33.
Pushed by ngraham into branch 'master'.

Fix currentIndex for auto-login user and session comboboxes

Fixed "user" combobox to match saved value.  
Fixed the "user" and "session" comboboxes to work with the "Reset" button.  
FIXED-IN: 5.26

M  +20   -3    src/package/contents/ui/Advanced.qml

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