E-commerce Product Page

Community feedback

• 

  Assurance Chioma Ikogwe•410

  @Aik-202Posted about 2 years ago

  Hi Mihir, this is wonderful! Nice work! I have some suggestions tho.

  1. Make sure to always put an alt text for all your images, I think that should fix most of your html and acessibility issues

  2. For the ones that say you should add img src, I think the problem as to do with the way you wrote the src for the images. It ought to have been ./images/img.jg as the images is in another folder.

  3. Use the main tag for the div with class modal as that is where your main content starts from.

  4. Instead of using strike tag, use this in your css : text-decoration: line-through

  5. I also noticed that once I change the number of shoes I'm ordering, it remains unchanged in my cart.

  6. Instead of using the button tag for your plus and minus icons, try using the div tag, and assigning the aria role button... i.e <div role = "button"> </div>

  Hope this helps.

  0

  ---

  

  MIHIR KUMAR•70

  @MIHI-botPosted about 2 years ago

  @Aik-202

  Thank you so much for your suggestion. I really didn't notice much about IMG tag attributes, both src, and alt, from now I will take care of these mistakes. "line-through" CSS property and role attribute are new to me and I am really thankful for your feedback. I will surely be better in the future from this feedback.

  0

  

  MIHIR KUMAR•70

  @MIHI-botPosted about 2 years ago

  @Aik-202

  Thank you so much for your suggestion. I really didn't notice much about IMG tag attributes, both src, and alt, from now I will take care of these mistakes. "line-through" CSS property and role attribute are new to me and I am really thankful for your feedback. I will surely be better in the future from this feedback.

  0

  

  Assurance Chioma Ikogwe•410

  @Aik-202Posted about 2 years ago

  @MIHI-bot Alright! I'm glad I was able to be of help

  1