Skip to content
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

Wrong option set in "Authentication required" modal #25247

Closed
wiswedel opened this issue Jan 21, 2021 · 2 comments
Closed

Wrong option set in "Authentication required" modal #25247

wiswedel opened this issue Jan 21, 2021 · 2 comments
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap design Design, UI, UX, etc. enhancement

Comments

@wiswedel
Copy link
Contributor

When an admin wants to perform sensitive actions and hasn't confirmed their password for some time, they get asked for their password just to confirm that action.

The password entry modal offers two buttons that don't make sense in this dialog:

image

Suggestion: Cancel/OK (default to OK)

@wiswedel wiswedel added enhancement design Design, UI, UX, etc. 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jan 21, 2021
@heyinsv
Copy link

heyinsv commented May 25, 2021

Based on user journey in this context, a simple Confirm button might do. And as far as Cancel action is concerned, the 'x' icon on top-right might do the work. Else that can be removed and a cancel button added beside Confirm button. A just did a small UI design for same.
Modal Auth-2

@szaimen
Copy link
Contributor

szaimen commented Jul 6, 2021

I think this was fixed in the meantime. Please reopen if you can still reproduce this on NC21.0.3 or higher. Thanks!

@szaimen szaimen closed this as completed Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap design Design, UI, UX, etc. enhancement
Projects
None yet
Development

No branches or pull requests

3 participants