Responsive QR-code-component

Community feedback

• 

  バレンタイン 😈•64,190

  @VCaramesPosted about 2 years ago

  Congrats @arshaqkv on completing this challenge!

  Card looks good.

  To clean up you HTML code, delete the <div class="card'>, its not needed, since you already have a parent container (<div class="container'>).

  Since this card is "card" you want to make your code semantically correct by change the <div class="container'> to an <article> element.

  Attached is a link with more detail.

  https://developer.mozilla.org/en-US/docs/Web/HTML/Element/article

  Marked as helpful

  0

  ---

  

  MUHAMMED ARSHAK K V•50

  @arshaqkvPosted about 2 years ago

  @vcarames Thank you for your feedback, So it is better to use semantic tags instead of just using the div tags

  0

  

  バレンタイン 😈•64,190

  @VCaramesPosted about 2 years ago

  @arshaqkv Its beneficial for SEO (Search Engine Optimization), screen readers and makes it a lot easier when reading code.

  Divs are for generic use. Mainly when there isn't a specific element for the content you want to wrap.

  Here's an article with more details on semantic HTML.

  https://developer.mozilla.org/en-US/docs/Web/HTML/Element

  0