Design comparison
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hi @berta-rf π, good job completing this challenge! π
I like this solution for the challenge. I have some suggestions you might consider to improve your code:
- In this challenge, the picture tag is not needed, since the image does not need to change depending on the viewport. This is necessary when the platform provides two or more images, for example, one for mobile and one for desktop. You can directly use the image tag in this solution.
- Since the images do not contribute anything to the content, and only serve as decoration, so it can be hidden for screen-readers if aria-hidden="true" is added and its alt attribute is left empty:
<img src="images/icon-music.svg" alt aria-hidden="true">
- In my opinion, the "Change" element should be a button, not a link, since it should perform an action, which is to change the plan and show the new plan's price. Use buttons to perform actions such as displaying a modal, performing calculations. Use anchor tags to redirect users to a part of the page or to an external site.
Please don't worry if your suggestions are long, they are just details. In the end, the project is well done π. Hope you find those tips helpful! π
Good job, and happy coding! π
Marked as helpful2@alvinjohnsonsoPosted almost 2 years agoHi @MelvinAguilar!
Thanks for taking the time to see my solution. I'm trying to familiarize myself working with HTML5 semantics and writing CSS. What you mentioned with the picture tag and the screen readers are a huge help to me. I'll definitely update the solution later on to apply the tips and suggestions you made π
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