----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124963/#review86678 -----------------------------------------------------------
Ship it! src/knewpasswordwidget.cpp (line 97) <https://git.reviewboard.kde.org/r/124963/#comment59664> this doesn't have to be done every time; do it in the constructor (or in the .ui file) src/knewpasswordwidget.cpp (line 101) <https://git.reviewboard.kde.org/r/124963/#comment59660> qBound(pwstrength, 0, 100) :-) src/knewpasswordwidget.cpp (line 146) <https://git.reviewboard.kde.org/r/124963/#comment59661> (This method is really a candidate for being unittested directly; just a thought) src/knewpasswordwidget.cpp (line 162) <https://git.reviewboard.kde.org/r/124963/#comment59662> use leftRef() instead of left() to avoid N string copies here. src/knewpasswordwidget.cpp (line 307) <https://git.reviewboard.kde.org/r/124963/#comment59663> + call update(), in case the widget is already shown? - David Faure On Oct. 10, 2015, 3:50 p.m., Elvis Angelaccio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124963/ > ----------------------------------------------------------- > > (Updated Oct. 10, 2015, 3:50 p.m.) > > > Review request for KDE Frameworks and Christoph Feck. > > > Repository: kwidgetsaddons > > > Description > ------- > > This widget is a stripped-down version of KNewPasswordDialog, without any > dialog-specific stuff. > > ### Why a new widget? > > This widget is meant to be easily embedded in any custom password dialog, > including KNewPasswordDialog. It's the least common denominator of features > that any password dialog should offer to the user. > > ### Features > > * Password visibility action (same as RR 124698). The password verification > field is hidden if the user shows the password. > * Background warning colour. The password verification field gets a coloured > background whenever the password and its verification don't match. (feature > borrowed from Keepass) > * Password status signal. This allows the upper level dialogs to update their > stuff (enable/disable OK button, show warnings for low password strength, > etc.) > * Password strength bar can be hidden. > * Unit test. > > ### Use cases > At least the following: > > * Ark (new archive dialog) > * KNewPasswordDialog refactoring (my next RR if this one gets accepted) > > > Diffs > ----- > > autotests/CMakeLists.txt ac708ef33e3be89db85d43911f96e536c52f741d > autotests/knewpasswordwidgettest.h PRE-CREATION > autotests/knewpasswordwidgettest.cpp PRE-CREATION > src/CMakeLists.txt e03e9bbd6d73811873b0a465f86da269f4295138 > src/knewpasswordwidget.h PRE-CREATION > src/knewpasswordwidget.cpp PRE-CREATION > src/knewpasswordwidget.ui PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/124963/diff/ > > > Testing > ------- > > > File Attachments > ---------------- > > knewpasswordwidget1.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/28/5796469c-28b7-4151-b9cb-6a327631db75__knewpasswordwidget1.png > knewpasswordwidget2.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/28/b3bfc9ac-ab8e-404c-8091-b5ad9e3e054f__knewpasswordwidget2.png > knewpasswordwidget3.png > > https://git.reviewboard.kde.org/media/uploaded/files/2015/08/28/dfaeea4e-65da-4961-b266-986a21f54ca7__knewpasswordwidget3.png > > > Thanks, > > Elvis Angelaccio > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel