Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Make auth validation less annoying #2539

Merged
merged 9 commits into from
Jan 31, 2019

Conversation

jryans
Copy link
Collaborator

@jryans jryans commented Jan 30, 2019

This is a quick pass to make authentication validation less annoying in the current world before fancy tooltips arrive.

Some highlights include:

  • Shake animation on error removed
  • All fields wait until blur before validating to avoid error spam as you type

Fixes element-hq/element-web#8293.

Watch as I tab through fields without anything scary happening:

tabbing-reg

@jryans jryans requested a review from a team January 30, 2019 20:57
@bwindels bwindels self-assigned this Jan 31, 2019
Copy link
Contributor

@bwindels bwindels left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just a few comments

src/components/views/auth/ServerConfig.js Show resolved Hide resolved
src/components/views/auth/ServerConfig.js Show resolved Hide resolved
src/components/structures/auth/Login.js Show resolved Hide resolved
@bwindels bwindels assigned jryans and unassigned bwindels Jan 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants