flex-box, grid-layout css, html

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

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

  • The main element should only ⚠️ be wrapping the card section since that is the main content of your page.

  • The article element is being used incorrectly ❌ and not needed for this challenge.

  • The “Reliable, efficient delivery Powered by Technology” is one single heading ⚠️ so the entire thing should be wrapped in a single h1 heading and either the top/bottom text will be wrapped with a span element.

  • Along with the blank alt tag, you also want to include the aria-hidden=“true” ⚠️ to your “icons” to fully remove them from assistive technology.

  More Info:📚

  MDN Aria-Hidden

  • There is no need for multiple stylesheets ❌ as it will affect performance, instead keep everything in one single stylesheet.

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

  Happy Coding! 🎆🎊🪅

  1

  ---

  

  Edwin Torrez•110

  @edwin08torresPosted almost 2 years ago

  @vcarames Thanks for the advice!!, yes, the multiple style sheet is because it has made it easier for me to use responsive, but you are right, performance could be affected, I still cannot understand media-querys exactly, but I will continue studying and reading , thank you, have a nice day, bro!.

  0