-
Notifications
You must be signed in to change notification settings - Fork 31
-
Notifications
You must be signed in to change notification settings - Fork 31
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
open sidebar menu over the content #100
Comments
I have mixed feelings about this. Part of me feels that people may use the left nav to help them find something and if it covers the content they may not be able to quickly find what they're looking for. I think let's leave it as it is currently until someone complains. |
I'm open for doing that 👍 I'd want to use shadcn's checkbox so it looks nice :) |
Do you want the checkbox to be visible when the menu is closed, with a tool-tip do i save user preference as a cookie or in local-storage? maybe we can use suitable icon / emoji instead of a checkbox? can you propose one? |
Yeah, I think an icon would be better for this as well. Local storage is a fine place for this. The more I think about it though, the more I think it's not necessary. Let's hold off. |
@kentcdodds
what do you think about this?
The text was updated successfully, but these errors were encountered: