QR

Community feedback

• 

  raya•2,870

  @rayaattaPosted 10 months ago

  Hello 👋sinth, congratulations on completing your first FED challenge 🎉

  I have some suggestions you might find interesting.

  1 Every html document should have a main tag that encloses the main page contents change <div class="card"> To <main class="card"> This will change nothing visually but it makes all the difference. You can learn more about the main tag in this article

  2 I noticed that you used <div id="bold"> for your heading. It is better if you replace it with <h1> Here's a quick guide on how to use them:

  The <h1> to <h6> tags are used to define HTML headings. <h1> defines the most important heading. <h6> defines the least important heading. Only use one <h1> per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with <h1>, then use <h2>, and so on. You can then style them in you css.

  Unlike what most people think, it's not just about the size and weight of the text It is about maintaining a clear and consistent hierarchy through out the document

  3 Make your html more semantic. divs do not carry any semantic value. They are used solely for decorations. You should replace <div id="passage"> with <p>.

  4 You can find out more about semantic Html and more in this article📑.

  I hope this helps 🙃

  Otherwise nice solution 👏

  0