Skip to content

add task solution#933

Open
Abdulahhh2005 wants to merge 9 commits intomate-academy:masterfrom
Abdulahhh2005:develop
Open

add task solution#933
Abdulahhh2005 wants to merge 9 commits intomate-academy:masterfrom
Abdulahhh2005:develop

Conversation

@Abdulahhh2005
Copy link
Copy Markdown

@Abdulahhh2005 Abdulahhh2005 commented Mar 11, 2026

Copy link
Copy Markdown

@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

to improve:

  1. these buttons should change photos
Image
  1. add the option to remove the product from the cart by pressing the button again
Image
  1. after selecting a product, the page should scroll upwards
Image
  1. slightly decrease the size of the hovered image because it now looks merged with the other one
Image
  1. page number should also be saved in the search parameters
Image
  1. after changing 'items on page', the page number should be set to 1
Image
  1. there is too much empty space
Image

Copy link
Copy Markdown

@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

last thing to improve:

  1. that button should also remove the product
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.

2 participants