After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 783236 - Sync up with password dialog in g-c-c
Sync up with password dialog in g-c-c
Status: RESOLVED FIXED
Product: gnome-initial-setup
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Initial Setup maintainer(s)
GNOME Initial Setup maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2017-05-30 09:27 UTC by Ondrej Holy
Modified: 2018-04-12 03:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Revert "password: visibly warn the user when the password is bad" (6.21 KB, patch)
2017-05-30 09:32 UTC, Ondrej Holy
committed Details | Review
password: Remove unused password hints (3.67 KB, patch)
2017-05-30 09:32 UTC, Ondrej Holy
committed Details | Review
password: Add strength indicator level for weak passwords (3.62 KB, patch)
2017-05-30 09:32 UTC, Ondrej Holy
committed Details | Review
password: Improve password hints (1.96 KB, patch)
2017-05-30 09:32 UTC, Ondrej Holy
committed Details | Review
password: Change colors of password strength indicator (2.73 KB, patch)
2017-05-30 09:32 UTC, Ondrej Holy
committed Details | Review

Description Ondrej Holy 2017-05-30 09:27:49 UTC
Bug 780002 changed strength indicator colors, password hints for weak passwords and others in g-c-c. It would be nice to sync up with it...
Comment 1 Ondrej Holy 2017-05-30 09:32:11 UTC
Created attachment 352856 [details] [review]
Revert "password: visibly warn the user when the password is bad"

This reverts commit 3049f2cc46e014aa7cac46a89ac94561c4249ced.

This was introduced when password strength check was not mandatory.
Comment 2 Ondrej Holy 2017-05-30 09:32:15 UTC
Created attachment 352857 [details] [review]
password: Remove unused password hints

Short password hints are not used for some time. Let's remove them.
Comment 3 Ondrej Holy 2017-05-30 09:32:20 UTC
Created attachment 352858 [details] [review]
password: Add strength indicator level for weak passwords

Add first level for short, or weak passwords to be obvious that
the strength indicator signalize something.
Comment 4 Ondrej Holy 2017-05-30 09:32:25 UTC
Created attachment 352859 [details] [review]
password: Improve password hints

Don't say "Good password!" for all acceptable password (e.g. weak passwords).
Say explicitely that password needs to be longer for short passwords.
Comment 5 Ondrej Holy 2017-05-30 09:32:30 UTC
Created attachment 352860 [details] [review]
password: Change colors of password strength indicator

Use red-yellow- green colors for strength indicator levels.
Comment 6 Rui Matos 2017-05-30 13:04:31 UTC
Review of attachment 352856 [details] [review]:

ok
Comment 7 Rui Matos 2017-05-30 13:05:42 UTC
Review of attachment 352857 [details] [review]:

sure
Comment 8 Rui Matos 2017-05-30 13:09:02 UTC
Review of attachment 352858 [details] [review]:

looks good
Comment 9 Rui Matos 2017-05-30 13:10:32 UTC
Review of attachment 352859 [details] [review]:

++
Comment 10 Rui Matos 2017-05-30 13:12:34 UTC
Review of attachment 352860 [details] [review]:

looks fine, thanks!
Comment 11 Ondrej Holy 2017-05-31 09:07:40 UTC
Attachment 352856 [details] pushed as 0c62c71 - Revert "password: visibly warn the user when the password is bad"
Attachment 352857 [details] pushed as 68cc39d - password: Remove unused password hints
Attachment 352858 [details] pushed as 2ca5f06 - password: Add strength indicator level for weak passwords
Attachment 352859 [details] pushed as 584a59d - password: Improve password hints
Attachment 352860 [details] pushed as 8d73189 - password: Change colors of password strength indicator
Comment 12 Michael Catanzaro 2018-04-12 03:44:49 UTC
Desynced a bit again, see https://bugzilla.gnome.org/show_bug.cgi?id=754213#c9 for details.