https://bugs.kde.org/show_bug.cgi?id=403366

Laurent Montel <mon...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/sdd
                   |                            |m-kcm/ed3decc06e387217bc131
                   |                            |8597ba1e97183d3b9b3
             Status|REPORTED                    |RESOLVED

--- Comment #2 from Laurent Montel <mon...@kde.org> ---
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

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to