----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126505/ -----------------------------------------------------------
(Updated Dec. 27, 2015, 2:40 p.m.) Review request for KDE Frameworks, KDE Usability, Christoph Feck, and David Faure. Repository: kwidgetsaddons Description (updated) ------- As discussed in RR 125619 and 126426, the password verification field (in a KNewPasswordWidget) should not be marked as "wrong" before the user even started typing the verification password. The revised approach is the following: * The user starts typing something as password, e.g. 1234 * The user types something else as verification password * As soon as the verification is not anymore a prefix of the password (e.g. verification = 122), the warning color is shown. * As soon as the verification is a prefix again (e.g. the user deletes the second 2, i.e. verification = 12) the warning color is not shown anymore. Diffs ----- autotests/knewpasswordwidgettest.h 43845128adec01aced4353c9f7986b7977829a2a autotests/knewpasswordwidgettest.cpp 297b88d5f18b9cd37f0d26d94e56f38870756f20 src/knewpasswordwidget.cpp a1b59454a2c2d7c09ac32acec52d3fffa73f77fc Diff: https://git.reviewboard.kde.org/r/126505/diff/ Testing ------- Autotests assert what described above. Gif pictures would explain the patch better than 1000 words, but I suck at creating them :( Thanks, Elvis Angelaccio
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel