Media queries

Community feedback

• 

  P

  Theo Harris•360

  @Theosaurus-RexPosted about 2 months ago

  Hi @Obiwumma,

  Your live site seems to differ from the design quite a bit. I'd recommend sticking with using just CSS until you are very comfortable, and then moving on to Tailwind, as Tailwind is a lot easier to use and a lot more effective when you have a strong grasp of what it's doing under the hood (it's all just CSS, after all!)

  Additionally, some notes on your HTML:

  • The first heading element in a document must ALWAYS be a <h1>, but you've used a <h2> for the product title.
  • I'd recommend letting the text wrap naturally rather than adding in explicit <br> tags in the middle of sentences. This will ensure a better experience for users across different device sizes (it doesn't have to match the text exactly as it appears in the design - it's more important that your site is usable)
  • I'd strongly recommend using an <img> tag for your image - I can see that you did that previously but commented it out, but that is the correct approach for this markup.

  0

  ---

  

  Obiwumma•110

  @ObiwummaPosted about 2 months ago

  @Theosaurus-Rex Thank you so much for the review. I'll stick with CSS from now on but I still don't see how tailwind is more effective

  1

  

  P

  Theo Harris•360

  @Theosaurus-RexPosted about 2 months ago

  @Obiwumma Sorry, to clarify - it's not more effective than CSS itself, but using it when you know CSS well is more effective than using it without strong fundamentals in CSS - I hope that makes sense! 😅

  Marked as helpful

  1