Skip to content

Develop#407

Open
protocolo23 wants to merge 5 commits into
mate-academy:masterfrom
protocolo23:develop
Open

Develop#407
protocolo23 wants to merge 5 commits into
mate-academy:masterfrom
protocolo23:develop

Conversation

@protocolo23

Copy link
Copy Markdown

@IanLuan IanLuan 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.

You’re almost there! Great progress so far, but there are a few small things missing to make it perfect:

  1. The nav list still has markers. Remove them using the list-style property.
  2. The logo in the header is too small. Adjust its size according to the Figma design.
  3. The sections are missing the vertical text on the left side.
  4. The images in the Shop section should have the same height.
  5. When hovering over the buttons, you add a border, which increases their height. Apply the border by default so the size doesn’t change on hover.
  6. The image in the Ingredients section should not be cropped on desktop, only on tablet and mobile.
  7. Remove the margin from the body so the sections can be full width.

@protocolo23 protocolo23 requested a review from IanLuan February 27, 2026 13:51

@IanLuan IanLuan 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.

Great job!! It's really good!

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.

2 participants