Responsive-Article-Preview-Card with HTML, CSS & JS

Community feedback

• 

  P

  Lo-Deck•2,790

  @Lo-DeckPosted about 17 hours ago

  Hi well done for this challenge,

  You need to use more semantic tag to improve your HTML, it's better for screen-reader and SEO.

  like:

  <div class="socials">
     <img src="./images/icon-facebook.svg" alt="">
     <img src="./images/icon-twitter.svg" alt="">
     <img src="./images/icon-pinterest.svg" alt="">
  </div>
  

  you can use ul and li.

  Use a div when there is no other choices, you can use section, article...

  It's better to use em or rem instead of px. FreeCodeCamp. FreeCodeCamp.

  Hope to be helpful.

  Marked as helpful

  1

  ---

  

  Efobi Francis•320

  @Efobi-FrancisPosted about 16 hours ago

  @Lo-Deck Thanks alot, i appreciate the feedback and it is very helpful. I'll be sure to implement the correction in future projects, if you have any other suggestions, please feel free to drop a feedback.

  once more Thanks.

  0