-
Notifications
You must be signed in to change notification settings - Fork 233
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
Figma #657
Figma #657
Conversation
Run & review this pull request in StackBlitz Codeflow. |
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
Let me know when you want me to merge. |
This is probably wider team decision. Please play with it and make your opinion what's the minimal state of changes we are OK to show to the world. |
I think that we're almost ready to merge. One more thing I would take care of - when you open the navigation menu on mobile devices and go to another page the menu stays open and I believe we should close it as a default behaviour. Screen.Recording.2023-04-20.at.12.22.45.mov |
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.
We can merge it!
The other tasks can be broken up into multiple issues and PRs.
@shinokada you do the honors.
Good job peeps! @jjagielka
Wait a minute. |
Done. |
Awesome! We're ready to merge @shinokada. |
Give me 10 min. |
Merged. |
π Description
The first draft of the new design.
β Checks
βΉ Additional Information