Design comparison
Community feedback
- @HassiaiPosted almost 2 years ago
Replace <a href="#" class="card__title"> with <h1> and <a href="#">with <p> to fix the accessibility issues. in the stylesheet, give them a cursor value of pointer when its hovered. e.g: h1:hover{ cursor: pointer color: hsl();}
Use rem or em as unit for the padding, margin and width for more on this watch this https://youtu.be/N5wpD9Ov_To
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful1@KevhecPosted almost 2 years ago@Hassiai Thanks for the suggestions, I'm going to watch the video and apply the changes specially the accessibility ones, I did something that worked but I think that your advices are a better way so I don't have to change the anchor tag's display.
0
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