D5312: KPasswordDialog: don't hide visibility action in plaintext mode

Fabian Vogt noreply at phabricator.kde.org
Thu Apr 6 07:02:12 UTC 2017


fvogt accepted this revision.
fvogt added a comment.
This revision is now accepted and ready to land.


  Tested locally, works fine!
  
  The behaviour is a bit unexpected, but IMO correct:
  
  - Enter a password
  - Make it visible
  - Delete it
  - Toggle visibility
  
  -> VisibilityAction stays visible, although input empty and password hidden

REPOSITORY
  R236 KWidgetsAddons

BRANCH
  kpassworddialog-leak

REVISION DETAIL
  https://phabricator.kde.org/D5312

To: elvisangelaccio, cfeck, fvogt
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170406/31189388/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list