Skip to content

Solution#998

Open
ExtymAndriy wants to merge 9 commits into
mate-academy:masterfrom
ExtymAndriy:develop
Open

Solution#998
ExtymAndriy wants to merge 9 commits into
mate-academy:masterfrom
ExtymAndriy:develop

Conversation

@ExtymAndriy
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Good progress, but you need to share your working DEMO LINK and pass all linter checks before requesting mentor's review

@ExtymAndriy
Copy link
Copy Markdown
Author

@ExtymAndriy ExtymAndriy requested a review from brespect May 20, 2026 19:04
Copy link
Copy Markdown

@brespect brespect left a comment

Choose a reason for hiding this comment

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

Good progress, let's start from Detail Page:

  1. Bread Crumbs icon is broken:
Image
  1. Why use should see the ID here?
Image
  1. Buttons are not aligned to left side and favourite icon is brroken:
Image
  1. This section looks completely different from the mockup, let's check Figma:
Image
  1. Cards in slider should be full view, not cropped:
Image
  1. Header should be same color as background:
Image

@ExtymAndriy
Copy link
Copy Markdown
Author

@ExtymAndriy ExtymAndriy requested a review from brespect May 21, 2026 12:38
Copy link
Copy Markdown

@2pasha 2pasha left a comment

Choose a reason for hiding this comment

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

good job! 👏

here are some improvements:

  1. content container should be the same width for all sections
Image
  1. it would be great if all images were +- the same sized
Image
  1. image looks broken
Image
  1. it will be great if user can delete item from cart by clicking there
Image
  1. fix images
Image Image
  1. footer should always be on the screen bottom
Image
  1. change favicon and title
Image
  1. fix layout inside cart
Image
  1. add total items count and take a look at amount in your header
Image
  1. implement some functionality for this button
Image

@ExtymAndriy ExtymAndriy requested a review from 2pasha May 22, 2026 15:14
Copy link
Copy Markdown

@Anton-Kuchmasov Anton-Kuchmasov left a comment

Choose a reason for hiding this comment

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

Almost done!

  1. When User selects any sorting parameters, they should appear in the URL as Search Params
Image
  1. Most of issues from prev mentor review did not fixed yet (favicon, non-existing icons, footer positioning etc.). Fix all of them, double-check is everything looks fine and then re-request review again
Image

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