-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Optical feedback for timeslider #2207
Comments
Yea this sounds good, I'd integrate it into #2070 which should solve a lot of load issues cc @marcelklehr |
Interesting challenge @seballot :D |
We just need to display a small spinner loader, should be easy, I can have a look |
Awesome tnx |
@seballot any progress on this one btw? :D |
mmmm, if I remember correctly I had a look, and could not reproduce the issue, even with big pads, when clicking to a revision the load of the new version was always very quick, so I did nothing... if you know where is the code which send the new text, you can do something like -> when sending the request, change css cursor property on body |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
What do you think about adding some type of optical feedback when browsing pad versions? On large pads (several thousand of versions) changing to another version might take some seconds, but the user don't know that the browser is currently calculating the pad text and might try to change to another pad version (in some cases changing pad versions doesn't work anymore and you need to reload). This leads to the problem that the user thinks that the timeslider is broken.
I think it would be a good idea to add a popup/overlay as long as the pad version is being calculated.
The text was updated successfully, but these errors were encountered: