Tech News Page - Using BS5 and no JS

Community feedback

• 

  バレンタイン 😈•64,190

  @VCaramesPosted almost 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • Unless your company requires you to use a library , I strongly suggest you stay away from using libraries until you fully grasp the fundamentals of HTML, CSS and JS. By using a library, you are robbing yourself from actually learning how to code.

  • It is best practice to use, Classes for your naming convention as classes are reusable, making them ideal for CSS styling. IDs on the other hand, are not reusable and are mainly used for JavaScript.

  • Unfortunately, JS is doing the opening and closing of your nav button. Since Bootstrap uses JS to make thing work (if you all the code that you added, the nav will still open and close).

  • There needs to be header, main and aside element in your HTML code to improve semantics and accessibility.

  • The "numbers" in the bottom are not headings the should be paragraphs.

  • The “images” in this component are purely decorative; They add no value. So their Alt Tag should be left blank and have an aria-hidden=“true” to hide them from assistive technology.

  • Implement a Mobile First approach 📱 > 🖥

  With mobile devices being the predominant way that people view websites/content. It is more crucial than ever to ensure that your website/content looks presentable on all mobile devices. To achieve this, you start building your website/content for smaller screen first and then adjust your content for larger screens.

  • You want to use variables for your colors.

  https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_custom_properties

  • I noticed that you went from two easy challenges to this one. I suggest, pacing yourself and slowing build your way up. This way, you can learn how everything things work in a smaller scale.

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding! 🍂🦃

  Marked as helpful

  0