Skip to content

Develop#412

Open
232567 wants to merge 4 commits into
mate-academy:masterfrom
232567:develop
Open

Develop#412
232567 wants to merge 4 commits into
mate-academy:masterfrom
232567:develop

Conversation

@232567

@232567 232567 commented Jun 28, 2026

Copy link
Copy Markdown

@DanielCamposDevX DanielCamposDevX left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The page looks good!
But we are missign some chekclists items:

  • The speed of animations is the same throughout the landing page (for example, increasing when hovering or moving blocks when scrolling)
  • Add favicon
  • Buttons “shop”, “to contact us”, “all products” must be with hover styles
  • All items in the shop section must have hover effects
  • All “learn more” buttons should also have hover styles

@232567 232567 requested a review from DanielCamposDevX June 30, 2026 20:50
@232567

232567 commented Jun 30, 2026

Copy link
Copy Markdown
Author

DEMO LINK.

@raulriato raulriato left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good job!

Comment on lines +65 to +79
&__card:hover {
transform: translateY(-8px);
box-shadow: 0 8px 24px rgba(52, 79, 16, 0.15);
}

&__image {
width: 100%;
height: 500px;
object-fit: cover;
transition: transform 0.3s;
}

&__card:hover &__image {
transform: scale(1.04);
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks a little strange. It would be better if you had one animation for the card.

@232567 232567 requested a review from raulriato July 4, 2026 10:17
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.

3 participants