Skip to content

Develop#402

Open
pzwolenik wants to merge 23 commits into
mate-academy:masterfrom
pzwolenik:develop
Open

Develop#402
pzwolenik wants to merge 23 commits into
mate-academy:masterfrom
pzwolenik:develop

Conversation

@pzwolenik

Copy link
Copy Markdown

No description provided.

@zibi-dev zibi-dev 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.

Hey, good job finishing this project! What I need now is a demo link to check your work.

@pzwolenik

Copy link
Copy Markdown
Author

Sorry, my bad!

DEMO LINK: https://pzwolenik.github.io/Eco_cosmetics/

@pzwolenik pzwolenik requested a review from zibi-dev September 4, 2025 14:39

@natalia-klonowska natalia-klonowska 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.

  • remove horizontal scrolling on mobile version
  • adjust burger menu icon (it's positioned too low)
  • reduce spacing between navigation and image
image
  • ensure menu extends to the full height of viewport:
image
  • shop should display different products when switching between tabs. you can create your own examples since design only shows 2 products for "Face" tab and 1 for "Hair" tab

  • It would be more intuitive to show product card page when clicking on a product image/title instead of using "all products" button

  • center shop title

image

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

Look good for me! Great job!

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.

4 participants