Responsive layout using CSS Flexbox and media query css

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted over 2 years ago

  I recommend you read up about

  • the picture element
  • headings for semantic meaning
  • how to indicate extra information for screenreaders when using strike through text (they will hear both prices at the moment - you need to make it clear that one is the old price)
  • how to write good alt text on images and how to treat decorative images

  Looking at this, perfume should not be a heading element, it should be a paragraph. If you want to give that ribbon text and heading some more affordance you can wrap them both in a hgroup element

  Images don’t need to be wrapped in divs every time you use them. I suggest using a basic css reset at the start of your style sheet and that would set images to display block, which is what you are trying to do I think.

  I hope these tips help

  Marked as helpful

  1

  ---

  

  Purna Chandra Saren•40

  @purna-devPosted over 2 years ago

  @grace-snow ok ma'am and thank you

  0