just used HTML and internal CSS

Community feedback

• 

  T

  Chamu•13,130

  @ChamuMutezvaPosted over 2 years ago

  Well done Bickey. Some things to consider are semantic elements such as main. The main element is considered important as it contains the main purpose of the site. In this case you can change this div class="QR-code" to the main element and the attribution div can be changed to the footer element.

  1

  ---

  

  Bickey kumar•30

  @BickeykrPosted over 2 years ago

  @ChamuMutezva thank you for your feedback. I will consider your suggestions & make changes.

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hay ! Good Job Bickey

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container. For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc . But use header tag only once in main heading element.

  Keep up the good work!

  1

  ---

  

  Bickey kumar•30

  @BickeykrPosted over 2 years ago

  @Crazimonk thank you, I will consider your suggestions & make changes.

  1

  

  Bickey kumar•30

  @BickeykrPosted over 2 years ago

  @Crazimonk
  hello there, can you help me here. if I change attribution div to footer element will it solve the accessibility issue which is in my solution.

  0