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

Made changes to the Navbar. #42

Closed
wants to merge 2 commits into from

Conversation

edidiesky
Copy link

@edidiesky edidiesky commented Jan 23, 2025

Closes #39

The navbar design and color tonality were not befitting for the product idea. So I had to improve on it by adjusting the with and padding of the navbar. The border and the border radius were also put into consideration. I also made changes in the framer-motion transition of the modal by removing the transition from the parent DIV to the child div. The modal layout was also modified and also the respective color tonality of the individual element of the modal.

Copy link

vercel bot commented Jan 23, 2025

@edidiesky is attempting to deploy a commit to the poulavbhowmick03's projects Team on Vercel.

A member of the Team first needs to authorize it.

@PoulavBhowmick03 PoulavBhowmick03 self-requested a review January 24, 2025 19:11
@PoulavBhowmick03 PoulavBhowmick03 self-assigned this Jan 24, 2025
@PoulavBhowmick03
Copy link
Contributor

@edidiesky Can you give me a recording of the change? the UI in my device looks distorted for some reason
thanks!

<MenuItem href="/" isActive={active === "home"}>Home</MenuItem>
{/* Add more menu items here */}
</div>
<div className="w-full px-6 md:w-[90%] max-w-[1000px] mx-auto border border-[rgba(255,255,255,.2)] rounded-full bg-gradient-to-b from-black via-gray-900 to-black py-4 min-h-[55px] flex items-center justify-between">
Copy link
Contributor

Choose a reason for hiding this comment

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

Keep the "StarkFinder" name and the Launch App only. And modify the popup for launch App to add the /chat/id, /transactions/id that is already present and add the /devx for the devxstark page to be accessed. Rest looks good!

@edidiesky edidiesky closed this by deleting the head repository Jan 25, 2025
@PoulavBhowmick03
Copy link
Contributor

@edidiesky You have closed the PR, is that itentional? Just making sure

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.

FRONTEND: Modify the current navbar of the landing page
2 participants