バレンタイン 😈• 64,190
@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
- The
mainelement should only ⚠️ be wrapping the card section since that is the main content of your page.
- The
articleelement 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
h1heading and either the top/bottom text will be wrapped with aspanelement.
- Along with the blank
alt tag, you also want to include thearia-hidden=“true”⚠️ to your “icons” to fully remove them from assistive technology.
More Info:📚
- 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
@edwin08torres
Posted
@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
