Skip to content

https://serhiydmytruk.github.io/react_phone-catalog/#903

Open
SerhiyDmytruk wants to merge 27 commits intomate-academy:masterfrom
SerhiyDmytruk:develop
Open

https://serhiydmytruk.github.io/react_phone-catalog/#903
SerhiyDmytruk wants to merge 27 commits intomate-academy:masterfrom
SerhiyDmytruk:develop

Conversation

@SerhiyDmytruk
Copy link

No description provided.

Copy link

@FaiHamid FaiHamid left a comment

Choose a reason for hiding this comment

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

Hi,
Run npm run deploy and add DEMO_LINK to pull request`s description.

@SerhiyDmytruk
Copy link
Author

@SerhiyDmytruk SerhiyDmytruk changed the title Attempt #1)) https://serhiydmytruk.github.io/react_phone-catalog/ Feb 24, 2026
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

  1. products in this section should not be discounted
Image
  1. slightly increase size of images when hovering them
Image
  1. these links should not lead to an error page
Image
  1. logo images look too small. also, it would be good if the page scrolled up after pressing them
Image
  1. after pressing the add button, redirect the user to the product page instead of adding the product to the cart
Image
  1. after changing the 'item on page' setting, the page number should be set to 1
Image Image
  1. product does not change when different colours are selected
Image
  1. fix image
Image
  1. make sure these numbers are the same
Image
  1. it would be good to remove the product from the page after pressing the 'Add' button again
Image

Copy link

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

To improve:

  1. Would you mind to make all cards height fixed? Now it looks sligthly weird in this block:
Image
  1. Click on one of those cards (see screenshot above) should scroll page up
  2. Height of heart & Add to cart buttons should be equal too:
Image
  1. When I click on Add to Cart button here, App opens full info page, but it expected adding to Cart only:
Image
  1. Check out
    https://serhiydmytruk.github.io/react_phone-catalog/#/phones/apple-ipad-pro-11-2021-1tb-spacegray

This memory option (as well as 1 or 2 TB) is available but after choosing it App redirected me to Not Found page

@SerhiyDmytruk
Copy link
Author

Other bugs I fixed!

**Note about "Add to cart" behaviour ** -- prev review: 4. When I click on Add to Cart button here, App opens full info page, but it expected adding to Cart only:

The behavior of AddToCart in Slider or on the category page will not be described in detail.

During the previous review, it was indicated that it is worth buying on the Product page - link. So that's what I did.

#903 (review)

5. after pressing the add button, redirect the user to the product page instead of adding the product to the cart

image

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.

almost done

  1. but I feel that you have misunderstood that point. that button should add the product to the cart, but in your implementation it redirects the user instead
Image
  1. make sure that these links do not lead to an error page
Image Image

@SerhiyDmytruk
Copy link
Author

Thanks for remarks!
1 - btn 'addToCart' now work as toggle.
2 - refactored links in the footer and at all

pls check.

thansk for explanation of issues

Copy link

@FaiHamid FaiHamid 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 just fix few details:

1.Pagination not adaptive
Image

2.After changing the sort field or the number of items per page, it should navigate to the first page
Image

@SerhiyDmytruk
Copy link
Author

Good remarks!

Copy link

@FaiHamid FaiHamid left a comment

Choose a reason for hiding this comment

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

Well done 👏

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