Skip to content
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: Phase 0 NavBar #4285

Closed
wants to merge 53 commits into from
Closed

feat: Phase 0 NavBar #4285

wants to merge 53 commits into from

Conversation

cbachmeier
Copy link
Contributor

@cbachmeier cbachmeier commented Aug 4, 2022

  • NavBar for token explore
  • Features old wallet
  • Hidded behind Feature Flag
  • Requires the correct env variables as mentioned in slack
  • There are a couple of difference between this NavBar and the one merged into interface-nft, notably some style changes as well as hiding NFT specific components
  • Code in the /nft/components folder are some shared wrapper components used across NFT design including an overlay, createPortal wrapper, and Box, Row, Column, which import vanilla extract sprinkles directly

@vercel
Copy link

vercel bot commented Aug 4, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
interface ✅ Ready (Inspect) Visit Preview Aug 15, 2022 at 8:23PM (UTC)

@cbachmeier cbachmeier marked this pull request as ready for review August 11, 2022 19:29
@vm vm self-requested a review August 11, 2022 19:31
@vm
Copy link
Contributor

vm commented Aug 12, 2022

This PR has a lot of features at once, could we split it up into...

  • Update the FeatureFlagModal vh
  • Add new Nav styled with no children
  • Add MobileNav conditional with no children
  • ChainSwitcher
  • Menu
  • SearchBar


if (!chainId || !info || !provider) {
// TODO: need to handle unsupported chain
if (!chainId || !info) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to render a state for unsupported chains

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants