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

fixing the position and adding animation to explore now button #560

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

s2sharpit
Copy link
Contributor

Description 📝

Closes: #549

Changes 💬

  • fixed Explore now button at center in bottom.
  • made it responsive for all screen sizes.
  • added ping animation

If something needs to be modified. Let me know.

@vercel
Copy link

vercel bot commented Jul 26, 2023

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

Name Status Preview Comments Updated (UTC)
gyanaguru ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2023 5:16am

Comment on lines +75 to +88
<div className="fixed bottom-8 z-40 left-1/2 -translate-x-1/2 grid place-items-center w-max">
<div className="absolute bg-black w-2/3 h-2/3 -z-10 rounded-full animate-ping"></div>
<Link href="/login"
className="flex items-center justify-center p-4 text-white duration-700 ease-out bg-black border-2 border-black rounded-full cursor-pointer w-fit h-fit dark:bg-zinc-50 hover:scale-110"
>
<h4 className="flex items-center text-xl font-archivo dark:text-zinc-800 space-x-3">
<span className="hidden lg:block">Explore Courses</span>
<span className="lg:hidden">Get Started</span>
<span className="text-white material-icons dark:text-black">
north_east
</span>
</h4>
</Link>
</div>

Choose a reason for hiding this comment

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

Instead of putting this directly in the index.tsx, I think we should create a component and then use that here. It would be more clean and maintainable in the long run.

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

Successfully merging this pull request may close these issues.

[Feat]: Update the position of Explore now button and add animation
3 participants