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

Méven Car <meve...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/use
                   |                            |r-manager/52ea204e39d5160d7
                   |                            |2066a573ec180baff4b0b63

--- Comment #7 from Méven Car <meve...@gmail.com> ---
Git commit 52ea204e39d5160d72066a573ec180baff4b0b63 by Méven Car.
Committed on 23/08/2019 at 14:23.
Pushed by meven into branch 'master'.

Replace the PasswordEdit field by a changePassword QPushButton, remove foreach
uses

Summary:

The current behavior to change password is quite unfriendly to say the least.
Clicking in a line field to open a dialog is not logical.
Replace this with a much simpler button leading to the dialog.

Test Plan:
Compile
kcmshell5 user_manager
Tried changing your password
Tried adding a user

Before:
{F7269649}

After:

{F7269690}

Reviewers: #plasma, ngraham, davidedmundson, sitter, #vdg, GB_2

Reviewed By: #plasma, ngraham, davidedmundson, #vdg, GB_2

Subscribers: nicolasfella, GB_2, plasma-devel

Tags: #plasma, #vdg

Differential Revision: https://phabricator.kde.org/D23337

M  +0    -1    src/CMakeLists.txt
M  +5    -13   src/account.ui
M  +7    -23   src/accountinfo.cpp
M  +2    -1    src/accountinfo.h
M  +2    -2    src/avatargallery.cpp
M  +2    -2    src/lib/accountmodel.cpp
M  +2    -2    src/lib/usersessions.cpp
D  +0    -32   src/passwordedit.cpp
D  +0    -39   src/passwordedit.h

https://commits.kde.org/user-manager/52ea204e39d5160d72066a573ec180baff4b0b63

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

Reply via email to