Interactive Rating Card using CSS Flexbox and JS
Design comparison
Solution retrospective
What might be a more optimal way to setup media queries when adjusting to smaller or bigger screens? I ended up just writing 5 different media queries to adjust the width of the main content when the screen would get smaller, but I feel like there is a better way to do that rather than writing so many queries
Community feedback
- @andreich1980Posted over 2 years ago
Hey Spencer.
First of all, the live website has scrollbars what isn't nice UX. Looks it is because your
.container
class has height of100vh
and then you have frontend mentor links below it.Other tiny problem is you didn't provide the solution URL in your README.md (should take us to this page)
Submit button is a div, but it should be
<button>
tag instead. This is what users expect to see.Other than that everything looks alright.
Cheers!
Marked as helpful0
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