-
Notifications
You must be signed in to change notification settings - Fork 438
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
When you enter the wrong username/password combination, the login dropdown turns blank #2515
Comments
The reason seems to be this |
@artlowel : Thanks for figuring this out. Assigning back to you for you or your team to work on. Also pinging @alexandrevryghem who added that code. |
I see the empty login dropdown very often after refreshing the page. I haven't yet figured out why, but suspect it happens when reloading too soon after restarting the backend or frontend in dev mode. I am used to shift-refreshing the home page to fix it... |
I've found that I'm able to reproduce this in Firefox but NOT in Chrome. In Firefox, I see the exact behavior shown in the description (entire login dropdown turns blank). In Chrome though, the login dropdown flashes blank but then shows the error message as expected. Nonetheless, it seems like it affects some browsers. I'll test the fix PR next and see if it resolves the issue for me in Firefox. |
@tdonohue ah that explains it. I use Firefox explicitly and see the blank login drop down so frequently that I was surprised nobody had noticed yet (it's certainly not only limited to a wrong username/password combination). |
Describe the bug
If you enter the wrong combination of username and password in the login dropdown, or on the login page. The component turns blank entirely. I can reproduce this on https://demo.dspace.org and on https://sandbox.dspace.org
To Reproduce
Steps to reproduce the behavior:
Expected behavior
It should show an error message and allow you to try again.
Related work
Possibly related to #2414?
The text was updated successfully, but these errors were encountered: