Frontend Mentor - Four card feature section solution

Community feedback

• 

  バレンタイン 😈•64,190

  @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 “Reliable, efficient delivery Powered by Technology” is one single heading so the entire thing should be wrapped in a single h1 heading along with a span element.

  • The intro heading and paragraph should be wrapped inside a header element.

  • The “icons” serve no other purpose than to be decorative; They add no value. Their Alt Tag should left blank and have an aria-hidden=“true” to hide them from assistive technology.

  More Info:📚

  https://www.w3.org/WAI/tutorials/images/

  • Using CSS Grid with Grid-Template-Areas will 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

  1

  ---

  

  Daniel Hernández•680

  @MarcosRubiPosted almost 2 years ago

  @vcarames Thanks for your tips, I have applied them to the code.

  1

  

  バレンタイン 😈•64,190

  @VCaramesPosted almost 2 years ago

  @MarcosRubi

  Glad I could help!

  Keep it up!

  0