Skip to content

Comments

React_Phone-Catalog#892

Open
ZAKANDRE wants to merge 22 commits intomate-academy:masterfrom
ZAKANDRE:develop
Open

React_Phone-Catalog#892
ZAKANDRE wants to merge 22 commits intomate-academy:masterfrom
ZAKANDRE:develop

Conversation

@ZAKANDRE
Copy link

No description provided.

Copy link

@Denys-Kravchuk9988 Denys-Kravchuk9988 left a comment

Choose a reason for hiding this comment

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

Please deploy the project and attach DEMO LINK

@ZAKANDRE
Copy link
Author

Please deploy the project and attach DEMO LINK

here is deployed project with my [DEMO LINK](https://zakandre.github.io/react_phone-catalog/ /)

Copy link

@etojeDenys etojeDenys left a comment

Choose a reason for hiding this comment

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

  1. page background looks broken
Image Image
  1. it would be nice if the product could be removed by pressing the button again
Image
  1. make sure you are increasing these numbers as well
Image
  1. it should be a link to the product page
Image

@ZAKANDRE
Copy link
Author

  1. page background looks broken

Image Image
2. it would be nice if the product could be removed by pressing the button again

Image 3. make sure you are increasing these numbers as well Image 4. it should be a link to the product page Image

I fixed all of this bugs

@ZAKANDRE ZAKANDRE requested a review from etojeDenys February 17, 2026 21:50
Copy link

@etojeDenys etojeDenys 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

right now, there is a small delay when changing the colour or capacity, so it is better to show a loader while the page is loading

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