Design comparison
SolutionDesign
Solution retrospective
Any feedback is welcome, thanks
Community feedback
- @vanzasetiaPosted about 2 years ago
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 thebody
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 helpful0@zousk-CamiloPosted about 2 years ago@vanzasetia thanks for the advice.......
0 - Firstly, remove all the alternative text from all icons or in other words, leave the
Please log in to post a comment
Log in with GitHubJoin 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