Design comparison
Community feedback
- @shashikantdev3Posted over 1 year ago
Hello there 👋. Good job on completing the challenge!
Here are some suggestions regarding your code that may interest you.
For Responsiveness
Your solution for desktop screen size looks excellent, but for Mobile screens, try to add more CSS to your media query. I suggest adding visually-hidden content for the price and oldPrice, so it doesn't generate an accessibility report and is also a best practice for accessibility.
Below is the link to my solution repo, please feel free to take a look and get the idea of proper restructuring of media queries, accessibility and usage of utility class and be careful when you use GRID and Flex.
You can see the tutorial, where Kevin explain all these things very clearly, below, is the YouTube link for the tutorial.
I hope you find it helpful!😏 Above all, the solution you submitted is 👌. 🎉Happy coding!
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