HTML, CSS - QR code component

Community feedback

• 

  Hassia Issah•50,410

  @Hassiai

  Posted about 2 years ago

  Replace <h4 class="title" > with <h1> to fix the accessibility issue.

  Every html must have <h1> to make it accessible. Always begin the heading of the html with <h1> tag wrap the sub-heading of <h1> in <h2> tag, wrap the sub-heading of <h2> in <h3> this continues until <h6>, never skip a level of a heading.

  To center .box on the page using flexbox, replace the height in component-box with min-height: 100vh.

  Give h1 and p the same font-size of 15px which is 0.9375rem, text-align: center, the same margin-left, margin-right and margin-top values. Give p a margin bottom value.

  For a responsive content, replace the width in .box and the img with max-width.

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  00

  Marked as helpful