Four Card Feature Section Using HTML and SCSS

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  Greetings, Cedrick Garcia! 👋

  Well done on this challenge! 👍 Your solution looks very nice and is responsive! 👏

  I'd like to suggest,

  • Using a single <h1> tag for the heading of the site, since it is really just one heading (not two separate headings). You can easily style the two parts of the heading using a <span> tag.
  • Removing cursor: pointer from the feature cards (since they aren't clickable elements).
  • Taking a look at your solution report and trying to clear up some of the errors that are there (it would be a good idea to simply use one <h1> tag on the page and use less important heading tags for the rest of the heading tags for SEO and semantic reasons). 😉

  Keep coding (and happy coding, too)! 😁

  Oh, and hey, if you found this comment helpful, an upvote would be appreciated! ;)

  3

  ---

  

  Cedrick Garcia•905

  @cedgarciaPosted almost 4 years ago

  @ApplePieGiraffe Thank you. Btw, I cant understand how to fix those HTML issues

  2

  

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  @cedgarcia

  Hey, Garcia! You can get rid of those HTML issues by using <h2> or <h3> tags for the headings of the feature cards (instead of using <h1> tags). It's considered a good practice to use only one <h1> tag on a page. Changing the tags of the other headings should be quick and easy, though. Good luck and have fun! 👍

  0