Four section card done using (CSS grid and flexbox)🤩😎

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey @Ljr777, some suggestions to improve you code:

  • The .heading-box should wrapped in a Header Element.

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

  • Reduce the padding from your Body Element; you should never use that much.

  • Your icons are overflowing when your screen shrinks. You either want to add a third layout to help keep them in place.

  Happy Coding! 👻🎃

  Marked as helpful

  0

  ---

  

  Ljr777•150

  @Ljr777Posted about 2 years ago

  hey @vcarames , how are you? thanks for the corrections I have made worked on them, but for the last one I dont know why my icons keep going out of the articles. Its happening at different screen sizes for me. can you please suggest to me a better way to position each icon inside each article. thank you so much for the feedback:D:D:D!!!

  0

  

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  @Ljr777

  You can give the icons a Position: Absolute and place them in the correct place.

  Here is how I did mine:

  https://vcarames.github.io/Four-Card-Layout/

  0

  

  Ljr777•150

  @Ljr777Posted about 2 years ago

  @vcarames ok thanks a lot, will study it and make corrections

  0