Product preview card

Community feedback

• 

  hitmorecode•7,240

  @hitmorecodePosted 3 months ago

  Nice looks good. A few tips

  • Don't set the image as background image. Both desktop and mobile version of the page use different images. In this case it's best to make use of the picture element. Just search for "html picture element" and you'll have enough examples on how to use it.
  • You should do a re-search on semantic html and try to use this approach more when writing html.

  I hope you find this helpful. Keep it up👌👍

  Marked as helpful

  0

  ---

  

  Otep02•220

  @Otep02Posted 3 months ago

  @hitmorecode thank you so much master, I will try this

  0
• 

  Vishal•260

  @vb8146649Posted 3 months ago

  This looks good , but the site isnt responsive and clips after certain width limit is reached and dont use margin in component else you should use padding for the body tag . Use <s> </s> tag to strike-through a text ,it may look the same but the screen readers and other accessibility software wouldn't announce the price is strike-through and will confuse the buyers as to why there are 2 prices .

  Marked as helpful

  0

  ---

  

  Otep02•220

  @Otep02Posted 3 months ago

  @vb8146649 this is noted sir, thank you so much for the suggestion.

  0

  

  Otep02•220

  @Otep02Posted 3 months ago

  @vb8146649 this is noted sir, thank you so much for the suggestion.

  0

  

  Otep02•220

  @Otep02Posted 3 months ago

  @vb8146649 this is noted sir, thank you so much for the suggestion.

  0

  

  Otep02•220

  @Otep02Posted 3 months ago

  @vb8146649 this is noted sir, thank you so much for the suggestion.

  0