@UDsGitHub
Posted
Hi Mehul,
Your solution looks great! one quick tip though would be to add cursor: pointer
to all your hoverable elements to make it more noticeable that these are interactable π. Just went through your report and noticed you have a lot of errors basically saying you need to add alt text into your images. I know sometimes we (me) assume we dont need alt text on some images that we believe dont contain any descriptive text, but we need alt text also for accessibility by for example those who use screen readers to access our sites. Also one of your html reports said you should enclose your content in landmarks: quick tip, try to use the html elements more often that divs that would also help with accessibilty by other software that use html structure to navigate through a site π. Before I leave ill give an example of the html stuff. you can use a main
element and add the class of card
you already have to itπ.
Marked as helpful
@Mehul9063
Posted
@UDsGitHub Thank you for your feedback. I will change my solution accordingly and would also implement the suggestions given by you in all my future projects.