Responsive layout built using CSS3, HTML5 and Tailwind CSS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted about 4 years ago

  Hey, Armando Guarino! 👋

  Good work on this challenge! Your solution looks good! 👏

  I suggest,

  • Using CSS background images to add and position the background SVGs in this project. That'll make your HTML cleaner and keep the background SVGs from getting in the way of the other elements on your page. Here's an article to get you started.
  • Only using the <h1> tag once in the card component (since there should really be only one main heading) and using less important heading tags (such as <h3> or <h4>) for the figures in the footer of the card.

  Keep coding (and happy coding, too)! 😁

  2

  ---

  

  Armando Guarino•540

  @guarmoPosted almost 4 years ago

  @ApplePieGiraffe thanks for the feedback! I will definitely use CSS background images instead of absolute positioning next time. It will save up some time and headaches! Regarding the <h1> tags, thanks for letting me notice it, I must have not noticed that I actually duplicated those!

  0