@MelvinAguilar
Posted
Hello there π. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- You should use an alt attribute empty for the images when they are used decoratively (Illustration of mockups). This practice avoids the reading of the description by screen readers, which can be bothersome for users with visual impairments.
- The element
<div class="btn">Register</div>
should be a link and not a div. Since the element has a hover effect and is interactive, it's better to use a interactive element like links or buttons for accessibility.
I hope you find it useful! π
Happy coding!
@dineshsutihar
Posted
Hello @MelvinAguilar ! π Thank you so much for your feedback and kind words! I truly appreciate your insights.
I'll definitely implement the alt attribute with an empty value for decorative images, as you suggested. Also, converting the <div class="btn">Register</div>
to a link makes a lot of sense for better accessibility. Your advice is valuable, and I'm committed to improving my coding practices.
Thanks again for taking the time to share your thoughts!π