Vanza Setia• 27,795
@vanzasetia
Posted
Greetings, Aayush Jaiswal! 👋
The following suggestions can help you to improve your solution.
- Alternative text for the logo should be 'Clipboard' instead of 'logo.'
- Alternative text for images should not contain any words related to "image" (such as picture, photo, logo, icon, graphic, avatar). It is already an image element (
<img>
) so the screen reader will pronounce it as an image. - Use
<a>
withdownload
attribute for the download buttons. - Not every image needs alternative text. Decorative images should not have alternative text (
alt=""
). This will tell the screen reader to skip over the image. As a result, it saves screen reader users time navigating the page. - Wrap the social media icons with
<a>
tags. Those are the social media links of the company. - Download the necessary icons instead of importing the whole Font Awesome icons to improve the performance of the site.
I hope my suggestions are useful. Happy coding! 😄
0
Aayush Jaiswal• 860
@aay7ush
Posted
@vanzasetia Thanks for your suggestions!
0
Vanza Setia• 27,795
@vanzasetia
Posted
@aay7ush
You are welcome! 👍
0