-
Notifications
You must be signed in to change notification settings - Fork 348
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix root password and LUKS passphrase visibility toggle (#1911360) #3119
Fix root password and LUKS passphrase visibility toggle (#1911360) #3119
Conversation
Anaconda supports showing a little "eye" icon in the password/passphrase entry field that can be used to make the entered text visible in plaintext during entry. For this to work correctly entry text visibility state needs to be set correctly. This has been broken by some of the password/passphrase entry related refactorings in the recent past, resulting in the "eye" icon not being visible. So put the initialization back for the root password and LUKS passphrase entries and also extend the docstring in the user configuration spoke to note how password visibility influences the toggle icon. Resolves: rhbz#1911360
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Thanks!
/kickstart-test --testtype smoke |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Anaconda supports showing a little "eye" icon in the password/passphrase
entry field that can be used to make the entered text visible in
plaintext during entry.
For this to work correctly entry text visibility state needs to be
set correctly. This has been broken by some of the password/passphrase
entry related refactorings in the recent past, resulting in the "eye"
icon not being visible.
So put the initialization back for the root password and LUKS passphrase
entries and also extend the docstring in the user configuration spoke
to note how password visibility influences the toggle icon.
Resolves: rhbz#1911360