STEPS TO REPRODUCE 1. open sddm_kcm, click "Advanced" tab 2. check "Auto-login" setting, apply the change and close the kcm 3. reopen sddm_kcm, uncheck "Auto-login" setting, close the kcm OBSERVED RESULT no apply/discard prompt appears when the kcm is closed EXPECTED RESULT apply/discard prompt shows up when you close the kcm after uncheck "Auto-login" without apply the change. SOFTWARE/OS VERSIONS Operating System: Arch Linux KDE Plasma Version: 5.14.90 KDE Frameworks Version: 5.54.0 Qt Version: 5.12.0 ADDITIONAL INFORMATION "Apply" button remains greyed out after you uncheck "Auto-login".
Can confirm on Neon 5.15.2 Nothing is saved and the apply button is disabled.
Git commit ed3decc06e387217bc1318597ba1e97183d3b9b3 by Laurent Montel. Committed on 23/03/2019 at 08:00. Pushed by mlaurent into branch 'master'. Fixed issue causing changed signal to be called with false argument Summary: AdvanceConfig::changed signal is called with changed=false arguement when QGroupBox::clicked is called with checked=false causing the Apply button to remain disabled. Test Plan: 1. Open SDDM settings 2. Enable auto-login, save and close. 3. Open SDDM settings again 4. Uncheck auto-login Apply button should now be enabled. Same steps for Relogin after quit checkbox. Reviewers: mlaurent Reviewed By: mlaurent Subscribers: mlaurent, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D19959 M +4 -2 src/advanceconfig.cpp https://commits.kde.org/sddm-kcm/ed3decc06e387217bc1318597ba1e97183d3b9b3
Git commit 0c76098e80c8ee61c7da9d85313461fc8a040651 by Albert Astals Cid, on behalf of Laurent Montel. Committed on 25/03/2019 at 22:58. Pushed by aacid into branch 'Plasma/5.15'. Fixed issue causing changed signal to be called with false argument Summary: AdvanceConfig::changed signal is called with changed=false arguement when QGroupBox::clicked is called with checked=false causing the Apply button to remain disabled. Test Plan: 1. Open SDDM settings 2. Enable auto-login, save and close. 3. Open SDDM settings again 4. Uncheck auto-login Apply button should now be enabled. Same steps for Relogin after quit checkbox. Reviewers: mlaurent Reviewed By: mlaurent Subscribers: mlaurent, plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D19959 (cherry picked from commit ed3decc06e387217bc1318597ba1e97183d3b9b3) M +4 -2 src/advanceconfig.cpp https://commits.kde.org/sddm-kcm/0c76098e80c8ee61c7da9d85313461fc8a040651