Mobile First Four Card Section using HTML and SASS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Greetings, Fidel Lim! 👋

  It's nice to see you complete another challenge! 😀 Good work on this one! 👍 Your solution looks great and responds well! 👏

  The only minor thing I suggest is using a single <h1> tag for the heading of the page (since it is really just a single heading, not two separate headings). You can easily style each part of the heading differently by wrapping one of them in a <span> tag. 😉

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  Fidel Lim•2,775

  @fidellimPosted over 3 years ago

  @ApplePieGiraffe Thankyou! yea true I could've done that too lol. Will definitely try to recall that if a similar case appears hahahah!

  0
• 

  Eric Salvi•1,350

  @ericsalviPosted over 3 years ago

  Great job with this one. Not a whole lot to give feedback on. The only thing I noticed is that the cards do not have a white background. Maybe implementing that would be the only thing needed?

  Some things you could try to implement for future solutions would be incorporating semantic HTML markup, https://www.w3schools.com/html/html5_semantic_elements.asp and the BEM naming convention?

  Keep up the momentum!

  1

  ---

  

  Fidel Lim•2,775

  @fidellimPosted over 3 years ago

  @ericsalvi thanks for the tip!

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  🌟🌟🌟🌟🌟 Looks Great... Very nice job!!

  1

  ---

  

  Fidel Lim•2,775

  @fidellimPosted over 3 years ago

  @palgramming thank you!

  0