Skip to content

245 home build restyle hero section#252

Open
nathandalpatan wants to merge 7 commits into
mainfrom
245-home-build-restyle-hero-section
Open

245 home build restyle hero section#252
nathandalpatan wants to merge 7 commits into
mainfrom
245-home-build-restyle-hero-section

Conversation

@nathandalpatan
Copy link
Copy Markdown
Contributor

i hard coded the images lmk if u want me to change it to payload data instead

@nathandalpatan nathandalpatan linked an issue May 7, 2026 that may be closed by this pull request
9 tasks
Copy link
Copy Markdown
Collaborator

@andrecamerino andrecamerino left a comment

Choose a reason for hiding this comment

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

Looking good! I've sent you some changes to your discord, here's a quick summary:

  • The size of the button should be all good for now, we can change that later
  • I think the photo background has some sort of greyed filter to kind of wash it out Check with kade first though
  • Are you able to add the white line at the bottom?
  • Are you all good to make that new collection?
  • I'll have a look at any carousel libraries

Copy link
Copy Markdown
Collaborator

@andrecamerino andrecamerino left a comment

Choose a reason for hiding this comment

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

Hi @nathandalpatan, almost there!

Logic looks great - just a couple more changes.

Ensure fully responsive

Image Image
  • Can we make sure that the size of the Hero content is responsive - especially with even larger screen sizes?
  • The content should be always centered (the group of components - not the text itself), and perhaps the white line at the botto should stretch as well? Ask Kade on #design.

Mobile CTA Button

Image
  • On smaller screen sizes, the button should not have an arrow (see Figma design for more details on the button)
  • This is because mobile does not use hover states

Just make these 2 changes and should be good to go!

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.

[Home] Build / Restyle Hero Section

2 participants