-
Notifications
You must be signed in to change notification settings - Fork 490
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
feat: navbar width toggle #710
Conversation
Click on the logo to collapse the side nav. Click again to get it back. License: MIT Signed-off-by: Oli Evans <oli@tableflip.io>
License: MIT Signed-off-by: Oli Evans <oli@tableflip.io>
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.
LGTM 👍
But it should have some kind of a hint to show that is possible to do this, or else it will be missed by most part of the users. Just something to think about 🙂
@fsdiogo strongly agree. I'll add something that appears when you hover over the nav. |
(in another PR tho) |
Awesome 👌 We can think of something cool 😎 |
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.
LGTM 😄 Just a little suggestion: why isn't the transition animated?
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.
Just found an issue:
This happens because we need to tell the size of the action bar. Could you just take a look at this, please?
If you're busy I can see if I can take care of it.
I'm merging this. I have a way to fix the size of the action bar |
Click on the logo to collapse the side nav. Click again to get it back. Invision design here: https://projects.invisionapp.com/d/main#/console/13924274/294213563/preview
(the weirdness with the light blue top bar is just an artefact from the gif screen recorder.)
License: MIT
Signed-off-by: Oli Evans oli@tableflip.io