-
Notifications
You must be signed in to change notification settings - Fork 116
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
sidebar animation improvement #43
Comments
Good point, I will also test this out. |
Hey, I was going to patch this out in |
sorry for the late reply, however, I couldn't find any issues with this? |
Do you have a powerful computer? Could be possible that the device I am using for testing is a bit slow. After some more testing, I have found the |
yeah, to say I have a powerful machine is an understatement, this is most likely a CSS performance issue since it changes the dom layout a lot during that animation |
however on my mid-range phone it doesnt seem to be an issue at all either, what browser did you use? |
It's good that you bring this up because on phones, the sidebar changes to the |
again, I'm not able to reproduce any of the issues you're having so idk how to help |
@halfmoonui , will using the |
the sidebar animation looks poor because of how much its moved offscreen, this can easily be fixed with:
The text was updated successfully, but these errors were encountered: