@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
-
Unless if 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.
-
There needs to be a
headerandmainelement in your HTML code. -
The “Reliable, efficient delivery Powered by Technology” is one single heading so the entire thing should be wrapped in a single
h1heading along 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. -
Using CSS Grid with
Grid-Template-Areaswill make things way easier when building the layout; it will give you full control of the layout.
Here is an example of how it works: EXAMPLE
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 🍂🦃
Marked as helpful
@gdemartino93
Posted
@vcarames Hi buddy boss, Im glad to receive your attention and your advice.
- I m using bootstrap to pratices it but im sure i can make this layout without BS
- I ve added header and main element
- I ve added aria-hidden="true" to the icons
About CSS Grid, I'm curious to ask you if isnt similar things as Bootstrap uses their grid layout 12col.
I look forward to your reply, Thanks for all.
