Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Task solution #457

Closed
wants to merge 1 commit into from
Closed

Task solution #457

wants to merge 1 commit into from

Conversation

valost
Copy link

@valost valost commented Jul 7, 2024

@valost
Copy link
Author

valost commented Jul 7, 2024

Hi! I just saw that the product details page doesn't work for some reason. I'll check what happened and fix it.

@valost
Copy link
Author

valost commented Jul 7, 2024

To be honest I am confused why this happened. I was fixing the lint errors and checking if the catalogue was still working. Everything seemed to work fine. Any ideas why this happened?

Copy link

@volodymyr-soltys97 volodymyr-soltys97 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 👍
Let's improve your page

  1. Add a favicon and actual title on the page
image
  1. Add cursor pointer for all clickable elements
image
  1. When we increase or decrease the product in the cart, the number in the top right next to the cart icon is not updated
image
  1. If you click on the product here, you get the following error
image image

@valost valost closed this by deleting the head repository Jul 8, 2024
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.

None yet

2 participants