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

Submit buttons showing ❌ on validation errors #9

Open
parkerdavis1 opened this issue Jul 12, 2023 · 3 comments
Open

Submit buttons showing ❌ on validation errors #9

parkerdavis1 opened this issue Jul 12, 2023 · 3 comments
Labels
P2 Cosmetic or nice to have

Comments

@parkerdavis1
Copy link
Collaborator

When logging in, if username/password is not correct, the Login button shows an ❌ emoji which persists after subsequently editing the username and password fields. This is visually confusing because the ❌ implies the Log in button is disabled when it is not.

image

The same thing happens when signing up and you enter an email that already has an account. I'm sure this happens on many different buttons throughout the app.

image

I would suggest removing the ❌ on buttons altogether.

@benhsm
Copy link
Collaborator

benhsm commented Jul 12, 2023

I'm not sure I agree with you on this one - the X is meant as feedback indicating that the action associated with the button has failed, where color and cursor shape on hover are used to indicate whether the button is enabled. In any case, thanks for raising the issue; I'm not going to make any changes to this component for the time being, but this is open to evaluation.

@parkerdavis1
Copy link
Collaborator Author

Exactly, the X indicates there is a problem associated with the button, when the problem is really with the input field(s). Alternative feedback could be red outlines on the offending input fields or an alert box with the error message (invalid username or password).

@gregv gregv added the P2 Cosmetic or nice to have label Jul 3, 2024
@gregv
Copy link
Contributor

gregv commented Jul 3, 2024

Was able to repro this on https://staging.trottrack.org/signup and login, we want the X to go away on the button since it also is reset with the red outline on the text boxes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Cosmetic or nice to have
Projects
None yet
Development

No branches or pull requests

3 participants