-
Notifications
You must be signed in to change notification settings - Fork 72
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: submenu dropdown #490
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Some details to change
components/UI/navbar.tsx
Outdated
@@ -247,6 +256,42 @@ const Navbar: FunctionComponent = () => { | |||
Domains | |||
</li> | |||
</Link> | |||
<Link href="/offers"> |
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 available yet so you need to delete that My Offers
href={`${ | ||
process.env.NEXT_PUBLIC_STARKNET_ID as string | ||
}/affiliates/individual-program`} | ||
> |
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.
Can you for an opening of the link in another tab instead of using href. Because people will go to another website and we also want them to stay on our website. (you can use window.open
or use other stuff within the Link
component).
You can do the same for all the other links
components/UI/desktopNav.tsx
Outdated
import { FaDiscord, FaGithub, FaTwitter } from "react-icons/fa"; | ||
import theme from "../../styles/theme"; | ||
|
||
const DesktopNav: FunctionComponent = () => { |
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.
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.
Last small bugs (small detail)
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
close: #472