Four card feature section using Flexbox

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

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

  • 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.

  • “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
• 

  Adriano•38,250

  @AdrianoEscarabotePosted almost 2 years ago

  Hello Danilo, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:

  <h1>Reliable, efficient delivery</h1>
  <h1>Powered by Technology</h1>
  

  The most appropriate in this case would be just an h1 tag! containing the two contents, to make them break a line, we can use a max-width, and for the styling we can use a span element with the content that will be changed!

  I noticed that the background colors of the project are not the same as the example layout, this is causing a lack of contrast on the four cards!

  The remainder is excellent.

  I hope it's useful. 👍

  Marked as helpful

  1