Review Request: kdepasswd KCM: add option to remove the user image

Ralf Jung ralfjung-e at gmx.de
Wed Mar 7 21:44:36 GMT 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104054/
-----------------------------------------------------------

(Updated March 7, 2012, 9:44 p.m.)


Review request for KDE Base Apps.


Changes
-------

Fixed whitespace issues


Description
-------

Currently it is possible to set a user image using the kdepasswd KCM, but it is not possible to remove that image again. This patch adds an appropriate option to the dialogue that pops up when clicking the current image.

It also adds "i18n" which obviously was forgotten for another button.


Diffs (updated)
-----

  kdepasswd/kcm/chfacedlg.h c389e49 
  kdepasswd/kcm/chfacedlg.cpp a80c5af 
  kdepasswd/kcm/main.cpp b664f98 

Diff: http://git.reviewboard.kde.org/r/104054/diff/


Testing
-------

Compiled, and verified that images are properly set up and removed again.


Thanks,

Ralf Jung

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120307/4cbb03df/attachment.htm>


More information about the kde-core-devel mailing list