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

Albert Astals Cid <aa...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://commits.kde.org/sdd |https://commits.kde.org/sdd
                   |m-kcm/ed3decc06e387217bc131 |m-kcm/0c76098e80c8ee61c7da9
                   |8597ba1e97183d3b9b3         |d85313461fc8a040651

--- Comment #3 from Albert Astals Cid <aa...@kde.org> ---
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

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

Reply via email to