Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

css grid

@zousk-Camilo

Desktop design screenshot for the Four card feature section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any feedback is welcome, thanks

Community feedback

Vanza Setia 27,795

@vanzasetia

Posted

Hello! 👋

Congratulations on finishing this challenge! 🎉

There are two things that can be done better.

  • Firstly, remove all the alternative text from all icons or in other words, leave the alt="" empty. It's because they are decorative images. It means that they don't have any useful information or if they don't exist, the users won't lose any information. Also, decorative images with alternative text can create unnecessary noise on the page.
  • Secondly, I recommend using rem for the body font size. This way, if the users change the font size through their browser settings, then the page will be able to adapt.

That's it! I hope you find this useful! 🙂

Marked as helpful

0

@zousk-Camilo

Posted

@vanzasetia thanks for the advice.......

0
Vanza Setia 27,795

@vanzasetia

Posted

@zousk-Camilo No problem!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord