-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Shadcn migration - desktop nav menu & search modal fixes #13647
Merged
Merged
Changes from 3 commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
f8e42bd
migrate nav to tailwind and new components
pettinarip c350ea3
convert eth home icon to svg
pettinarip 4153a1f
style desktop nav menu using tailwind classes
pettinarip 36fe6f7
Merge branch 'dev' into shadcn-nav
pettinarip 35370d8
set a default bg color to the menu content to avoid flashing color fe…
pettinarip 9e4b946
define imports outside of component declaration
pettinarip f365795
refactor nav menu variants and fix gap in active items
pettinarip 480ed43
refactor search logic
pettinarip 4bf7998
Merge branch 'dev' into shadcn-nav
pettinarip 720e267
Merge branch 'dev' into shadcn-nav
pettinarip 1ee1ae5
Merge branch 'dev' into shadcn-nav
pettinarip a644466
fix extra padding in opened submenus
pettinarip e39e7bc
fix styles on search button hover effect
pettinarip 5311011
fix hover color on the mobile hamburger button
pettinarip 2f80308
Merge branch 'dev' into shadcn-nav
pettinarip 154bc3c
update menu colors to use hsla
pettinarip File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Not at all against this, but just noting an alternative way this can be done as we migrate to TW:
Can see where both approaches could come in handy, not sure I have a strong preference or desire to standardize it.
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.
Great idea! I haven't thought about that option, I like the no-js dependency. I'll test it out and let you know.