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

UX issue: Two clicks required to sign in after failing to sign in with full MXID due to CORS #8412

Closed
dkasak opened this issue Feb 5, 2019 · 3 comments

Comments

@dkasak
Copy link
Member

dkasak commented Feb 5, 2019

  • Browser: Firefox 65
  • OS: Arch Linux

Description

Somewhat related to #7845 (the reproduction steps are the same). After sign-in fails with the full MXID due to CORS issues (#7875) and changing the Username to just the localpart to work around this, the Sign in button requires two clicks to work instead of just one.

Steps to reproduce

  1. Visit https://riot.im/develop/
  2. Click on Advanced/Other "Find other public ..."
  3. Focus the Homeserver URL input box.
  4. Delete matrix.org, type in my HS domain.
  5. Focus Username input box and enter full MXID.
  6. Focus Password input box and enter password.
  7. Click on Sign in; this fails due to CORS errors (expected).
  8. Focus Username input box again and edit it to contain only the local part of the MXID.
  9. Without clicking anywhere else, click on Sign in.

Expected behaviour

Riot signs in.

Actual behaviour

Riot ignores the click. It only signs in after an additional click on Sign in.

@dkasak dkasak added the redesign label Feb 5, 2019
@aaronraimist
Copy link
Collaborator

aaronraimist commented Feb 20, 2019

Hmm I cannot reproduce this on the current version of /develop.

@jryans
Copy link
Collaborator

jryans commented Feb 20, 2019

@dkasak Does it also seem fixed on /develop for you as well?

@dkasak
Copy link
Member Author

dkasak commented Feb 20, 2019

Yes, I can't reproduce it anymore either.

@dkasak dkasak closed this as completed Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants