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

Initial commit #525

Closed
wants to merge 2 commits into from
Closed

Initial commit #525

wants to merge 2 commits into from

Conversation

OPTIMISTIXX
Copy link

@OPTIMISTIXX OPTIMISTIXX commented Oct 30, 2024

Copy link

@VitaliyBondarenko1982 VitaliyBondarenko1982 left a comment

Choose a reason for hiding this comment

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

Good start.
Looks like project doesn't finished yet.

  • no any gadjets on phone, accessories, tablets pages
    Screenshot 2024-10-31 at 09 14 21

  • in some places font doesn't match to design

  • also many places where layout doesn't match to design. for example
    Screenshot 2024-10-31 at 09 13 50
    Screenshot 2024-10-31 at 09 14 48

  • layout broken for mobile on product details page for mobile, also check main slider. and other pages
    Screenshot 2024-10-31 at 09 15 39
    Screenshot 2024-10-31 at 09 16 23

  • in cart see delete button. some redundant char hide remove button
    Screenshot 2024-10-31 at 09 18 28

  • amount of items in header and cart should be the same
    Screenshot 2024-10-31 at 09 18 15

  • product card on the home page doesn't have border, favorite button doesn't match to design, on hover image cutted
    Screenshot 2024-10-31 at 09 17 02
    Screenshot 2024-10-31 at 09 17 40

  • also check other places in project...

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