-
Notifications
You must be signed in to change notification settings - Fork 0
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: Create Hero Section #10
base: main
Are you sure you want to change the base?
Conversation
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.
src/app/components folder should be in src/components folder
src/app/components/HeroSection.tsx
Outdated
</div> | ||
<div className="flex flex-col md:flex-row justify-center gap-4 text-[1.125rem]" ref={animateContainerRef}> | ||
<button className="px-5 py-4 bg-[var(--color-white)] text-[#010840] rounded-xl"> | ||
<a className="flex justify-center items-center gap-2" href="">Wypróbuj <ArrowUpRight size={32} color="#010840" /></a> |
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.
As we use NextJS we should use NextJS Link component for navigation imo
src/app/components/HeroSection.tsx
Outdated
<p className="text-[clamp(1.2rem,2cqw,1.75rem)] max-w-screen-md mx-auto pt-8" > Nasz system rekomendacyjny dopasuje go do tytułu lub opisu twojej pracy dyplomowej. </p> | ||
</div> | ||
<div className="flex flex-col md:flex-row justify-center gap-4 text-[1.125rem]" ref={animateContainerRef}> | ||
<button className="px-5 py-4 bg-[var(--color-white)] text-[#010840] rounded-xl"> |
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 we use shadcn components for things that shadcn has comoponents for?
src/app/globals.css
Outdated
--color-white: #E7EAFB; | ||
--color-primary: #5F7ECD; |
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.
I think that if we will move these to tailwind.cofnig.ts it will be easier to use in code
package.json
Outdated
@@ -11,6 +11,7 @@ | |||
"dependencies": { | |||
"class-variance-authority": "^0.7.0", | |||
"clsx": "^2.1.1", | |||
"gsap": "^3.12.5", |
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.
I think it will be better to first create app without any animations and then add them
…o tailwing.config
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.
If you need any help in resolving conflicts during merging, please text me
No description provided.