Develop

[Patryk-Buczkowski]

• DEMO LINK

[nataliaklonowska]

It's a good start, so re-request a review when you have a finished part that you want to be checked :)

[Patryk-Buczkowski]

Hello, here is some things to fix:

1. Recommended products not shows at accessories product cards
   2.Center your h1,h2, I think its will looks better, 3.If you want you can add your theme style button to header 4.Make changes in your dark theme its looks bad
   5.At some products cards if you choosing options like color or capacity, web site stops work(accesories)
   6.When you adding products in product cart (clicking + or - button) it starts changing position.
   7.Make contacts page with your info

Can You show me point 6?

[natalia-klonowska]

point 6 meant that when you change amount of products in cart it's value change and with it it's width and it affect the position what you can see even for number 2 but it's more visible e.g 100:

[Patryk-Buczkowski]

Contacts i will add for sure when i create account on linkedin.

[natalia-klonowska]

resolve all comments below

[natalia-klonowska]

• center page content
  

[natalia-klonowska]

• you should have max width at 1200px for all pages
  
  

[natalia-klonowska]

• you should be able to remove product from basket by clicking button again
  

[natalia-klonowska]

• To return to the previous page, you need to use the back button twice and browser don't remember more than last visited page
• on any product page it's name should be active in breadcrumbs
  

[natalia-klonowska]

• all clickable elements should have pointer and selected elements should be marked
  
  
  

[natalia-klonowska]

• make sure to always have some elements on recommend section (if list is empty have some default)
  

[natalia-klonowska]

when product color name consists of more than one word and you try to change capacity, wrong word is changed and page can't load (and if this happens it should redirect to a 'Page Not Found' instead)

[Patryk-Buczkowski]

on figma in breadCrumb product name is grayedout

[Zibi95]

• There is to little space between logo and navbar
  
• there is mismatch in colors of the titles
  
• Banner images change when you hit mobile breakpoints (don't know if this is intended)
  
• Slider overflows that 1200px of the layout
  

[Zibi95]

when you hit 1200px breakpoints weird stuff happens on category section

[Zibi95]

The banner slider looks to small when compared to the big title

[Patryk-Buczkowski]

Banner images change when you hit mobile breakpoints (don't know if this is intended)
yes, it is ;)

[Zibi95]

• Those buttons should have the same height as images also i think that the radius is to big and those buttons doesnt fit
  

• You should be able to decide how many cards you show on given breakpoint and then lower it down as you screen shrinks
  

• These gaps are huge
  

[Zibi95]

• on every view max width of the view i set to 1136px for desktop
  

[Zibi95]

Those arrows in breadcrumbs are huge compared to other elements also color is different then on designs