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
[DO NOT MERGE] feat: navigation light mode #2730
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
52cba3c
to
f2fcc21
Compare
size-limit report 📦
|
4785d52
to
bf171e5
Compare
e9498cb
to
bd10031
Compare
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.
I am noticing a few minor differences from the design, I don't know how much we want to nitpick them though. Any thoughts @damann?
- There's a difference in the green colour used for the environments.
- The selected state is currently "inside" the hover state background, in the design it lives below it.
- The burger menu icon seems smaller in the design.
packages/components/navbar/src/NavbarSwitcher/NavbarSwitcher.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Kristoffer <denkristoffer@users.noreply.github.com>
* feat: introduce new notification 'info' variant for NavbarAccount * chore: bump version
ad06143
to
d1a6baf
Compare
7d0f364
to
d1a6baf
Compare
* feat: set maxWidth for navbar * feat: enable wide and fullscreen variants * chore: remove unused import * feat: only limit the content width not the components
Purpose of PR
Change the color schema of the Navbar component from dark to light. This is not a new variation but a complete switch of style. The overall structure and layout of the component is unchanged.
But some of the classNames are changed in order to be more meaningful.