test_for_mentor

[zipic]

https://zipic.github.io/react_phone-catalog

[TarasHoliuk]

How does it look now:

And 404 page on any link click

[TarasHoliuk]

Please, make sure the App works as expected
If you have any questions feel free to ask in the chat or join to Q&A session

[TarasHoliuk]

I'd recommend you use HashRouter in this case

[TarasHoliuk]

There should be more Routes. Also, you have to cover paths to a separate product, for example: /phones/somePhoneID

[zipic]

please not review

[IvanFesenko]

Content doesn't centered. There are no mobile/table version
https://github.com/mate-academy/react_phone-catalog/assets/61904995/436639f8-09be-4230-8204-b78c2d99c213

Border is gone some how

Imposible to change value. Page not havee 100% width
https://github.com/mate-academy/react_phone-catalog/assets/61904995/616611b1-1eed-4b1f-b772-0db5fdb5c1a2

add favIcon

Impossible to open phone page
https://github.com/mate-academy/react_phone-catalog/assets/61904995/37a82397-1f2f-4ebd-bce3-3aecb09000bd

[TarasHoliuk]

1. Add favicon

2. The image on the product page looks suboptimal:
   

3. When clicking on different colors link is broken (capacity links are broken initially):
   

4. The header/page is still not full-width. The search looks not good. The selected tab is not highlighted.
   

5. Can't remove this item from cart (also, may sure it works fine on the favorites page):

6. Typo:
   

7. Footer always should be at the bottom of the page:
   

[TarasHoliuk]

GJ 👍
But there are things that should be fixed (check comments)

[lerastarynets]

Pls watch the video review via the link

[IvanFesenko]

Pagination issue - you should reset selected page in case Items per page change
https://github.com/mate-academy/react_phone-catalog/assets/61904995/0bef21e5-346a-47eb-9130-0491108269d3

[IvanFesenko]

well done