News Homepage using Bootstrap5

Community feedback

• 

  Travolgi 🍕•31,280

  @denielden

  Posted over 2 years ago

  Hello Nitin, You have done a good work! 😁

  Some little tips to improve your code:

  • add header tag and wrap the navbar for improve the Accessibility
  • add main tag and wrap the main page content for improve the Accessibility
  • also you can use article tag instead of a simple div to the container of news for improve the Accessibility
  • add transition on the element with hover effect
  • instead of using px use relative units of measurement like rem -> read here

  Keep learning how to code with your amazing solutions to challenges.

  Hope this help 😉 and Happy coding!

  12

  Marked as helpful
• 

  Adriano•42,890

  @AdrianoEscarabote

  Posted over 2 years ago

  Hi Nitin Singh, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

  wrap the nav tag with the header tag:

  <header>
  <nav class="navbar navbar-expand-lg navbar-light">
  </header>
  

  wrap this div:

  <div class="container-lg homa-page mt-lg-1">

  with the main tag!

  I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written Read more, he is not confirming a form, or something like, it will be redirected to another page, to read more about!

  To resolve this:

  <a class="readmorebtn btn px-4 mt-2">R E A D M O R E</a>

  The rest is great!

  I hope it helps... 👍

  00

  Marked as helpful
• 

  Nitin Singh•60

  @Nitinsingh08

  Posted over 2 years ago

  Thank you so much for your suggestion , I will surely implement them in the next project. The fact that someone is reviewing my project and helping is too much for me ,again , thanks for this .

  00