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 706067 - revert password generation feature
revert password generation feature
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: User Accounts
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Ondrej Holy
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-08-15 13:14 UTC by Ondrej Holy
Modified: 2013-08-16 21:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
revert password generation to the password dialog (5.80 KB, patch)
2013-08-15 13:16 UTC, Ondrej Holy
reviewed Details | Review
add password generation to the add user dialog (3.82 KB, patch)
2013-08-15 13:16 UTC, Ondrej Holy
reviewed Details | Review
user-accounts: Add password generation back to the password dialog (5.17 KB, patch)
2013-08-16 21:09 UTC, Stef Walter
committed Details | Review
user-accounts: add password generation to the add user dialog (3.00 KB, patch)
2013-08-16 21:09 UTC, Stef Walter
committed Details | Review

Description Ondrej Holy 2013-08-15 13:14:35 UTC
Password generation feature was removed for 3.8, however there are wishes to be reverted.
Comment 1 Ondrej Holy 2013-08-15 13:16:28 UTC
Created attachment 251732 [details] [review]
revert password generation to the password dialog
Comment 2 Ondrej Holy 2013-08-15 13:16:55 UTC
Created attachment 251733 [details] [review]
add password generation to the add user dialog
Comment 3 Stef Walter 2013-08-16 21:09:06 UTC
Created attachment 251952 [details] [review]
user-accounts: Add password generation back to the password dialog
Comment 4 Stef Walter 2013-08-16 21:09:11 UTC
Created attachment 251953 [details] [review]
user-accounts: add password generation to the add user dialog
Comment 5 Stef Walter 2013-08-16 21:09:22 UTC
Review of attachment 251733 [details] [review]:

::: panels/user-accounts/um-account-dialog.c
@@ -285,3 @@
-        if (strlen (verify) == 0) {
-                gtk_widget_set_sensitive (self->local_verify, strength_level > 0);
-        }

Again, removed this. I took it out in a previous patch. It made <TAB> key unusable in the dialog.

@@ +462,3 @@
 
+static void
+activate_icon (GtkEntry             *entry,

Fixed this function name.
Comment 6 Stef Walter 2013-08-16 21:09:31 UTC
Review of attachment 251732 [details] [review]:

Looks good with the change below.

::: panels/user-accounts/um-password-dialog.c
@@ -91,3 @@
-        if (strlen (verify) == 0) {
-                gtk_widget_set_sensitive (um->verify_entry, strength_level > 0);
-        }

Already removed this in another patch as it made <TAB> key unusable in the dialog, so removing here again.

@@ +421,3 @@
 
+static void
+activate_icon (GtkEntry             *entry,

Fixed this function name.
Comment 7 Stef Walter 2013-08-16 21:10:24 UTC
Attachment 251952 [details] pushed as a3d0840 - user-accounts: Add password generation back to the password dialog
Attachment 251953 [details] pushed as 60e1999 - user-accounts: add password generation to the add user dialog