-
Notifications
You must be signed in to change notification settings - Fork 903
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
Adds loading indicator while fetching history #3874
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great.
I am wondering if we'd want to still show the name of the zone while loading? I think it's nice small feedback that the users know they have clicked the right zone and is now awaiting the detailed data.
(I realize it's less than a second usually)
Adds a bit of extra work, so regardless we could address that in another PR
Good point @Kongkille, I have changed it to this now: I also fixed the back-arrow button being white on hover so you couldn't see it 🤔 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent!
Description
Makes it clear to users that data is being fetched, instead of first showing "no data is available" until data for a zone is loaded.
Before
After