HTML, CSS flex

Community feedback

• 

  Ecem Gokdogan•9,380

  @ecemgoPosted over 1 year ago

  Some recommendations regarding your code that could be of interest to you.

  In order to fix the accessibility issues:

  • You need to replace <div class="main-content"> with the <main class="main-content"> tag. You'd better use Semantic HTML, and you can also reach more information about it from Using Semantic HTML Tags Correctly.
  • Each main content needs to start with an h1 element. Your accessibility report states page should contain a level-one heading. So, you need to use a <h1> element in the <main> tag instead of using <h6>. You can replace your <h6>Improve your front-end skills by building projects</h6> element with the <h1>Improve your front-end skills by building projects</h1> element.

  Hope I am helpful. :)

  1

  ---

  

  Özcan ÇAVGA•20

  @OzcanCAVGAPosted over 1 year ago

  Thank you @ecemgo

  1
• 

  Anisha Murthy•60

  @anishamurthyPosted over 1 year ago

  Hi, ensure you add a main tag to your HTML and don't use an h6 tag, start with h1 instead. the h1 tag is the most important heading on the page. You should add alt text to your image by mentioning the URL the QR code links to, for example, QR code for the URL(insert URL). Alt text should always be added to non-decorative images that are meaningful to the page. You can add a CSS reset of your choice as well, for example, https://andy-bell.co.uk/a-modern-css-reset/. You can use a code formatter like prettier. Congratulations on completing the challenge you submitted a great solution!

  1

  ---

  

  Özcan ÇAVGA•20

  @OzcanCAVGAPosted over 1 year ago

  thank you for your suggestions@anishamurthy

  0