Design comparison
Solution retrospective
Feedbacks are always welcome :)
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hello there ๐. Good job on completing the challenge !
I have some feedback for you if you want to improve your code.
- Remove the
min-width: 100vw;
property from the.card-container
selector as this generates a horizontal scrollbar on some devices.
-
Some images do not need to have a descriptive alt attribute, you can leave the alt text of the music icon and background empty. This will hide it from some technologies such as screen readers.
The main reason for hiding an image from screen readers is that image can be used for decorative elements and may not provide any meaningful information to users of assistive technology. Hiding them from screen readers can prevent confusion and reduce the amount of unnecessary information that is read to users.
You can read more about this here ๐
I hope you find it useful! ๐
Happy coding!
Marked as helpful1 - Remove 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