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

Beta: The loading pulsating effect is incredibly distracting #201

Closed
person4268 opened this issue Sep 26, 2024 · 4 comments · Fixed by #213
Closed

Beta: The loading pulsating effect is incredibly distracting #201

person4268 opened this issue Sep 26, 2024 · 4 comments · Fixed by #213
Labels
enhancement New feature or request

Comments

@person4268
Copy link
Contributor

person4268 commented Sep 26, 2024

Is your feature request related to a problem? Please describe.
I've just tried the beta, and while I like it overall, the pulsating effect on the left when keys are loading is distracting me pretty badly, especially when I've already grabbed the things I need and I'm trying to focus on the main view on the right. Additionally, each individual instance of the effect desyncs, making the effect even worse. This isn't the most significant issue, but it is genuinely throwing me off a fair bit.

Describe the solution you'd like
Replacing the effect with something else. I'm not sure exactly what, but preferably something that isn't animated and is fairly discreet.

Describe alternatives you've considered
Slowing down and/or making sure the pulsating in sync would help, but I just find the effect intrinsically distracting when I'm trying to find a key in the tree or whatnot.

Additional context

(ignore me realizing everything had already loaded at the beginning)

2024-09-26.09-25-00.mp4
@person4268 person4268 added the enhancement New feature or request label Sep 26, 2024
@jwbonner
Copy link
Member

The main requirement is that it needs to be very obvious to the user that something is happening, since otherwise it looks like the data isn't available. Below is a slightly more subtle option that uses the cursor and an opacity shift to indicate that progress is being made.

If this looks better to you, we can try it in the next beta and get feedback from more users.

Screen.Recording.2024-09-26.at.1.34.59.PM.mp4

@person4268
Copy link
Contributor Author

Yeah, that's a lot better and more intuitive imo. Signals pretty clearly that "these aren't usable" (yet) while I couldn't get an instant read on what the original effect was trying to indicate for a minute. Thanks for fixing that. My other qualms are very minor and almost nitpicky and fairly subjective in comparison, but I can write them out if you want them.

@jwbonner
Copy link
Member

I’d welcome hearing any feedback you have, and we can figure things out if some are subjective.

@jwbonner
Copy link
Member

jwbonner commented Oct 5, 2024

Feel free to open a new issue if you have additional feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants