Design comparison
Solution retrospective
Still having trouble making all of the text white on radio button when hover.
Community feedback
- @Deevyn9Posted about 2 years ago
Hi Arthur, congrats on completing this project, It looks really incredible and it works great too, you can make it better by swapping the background color of the radio buttons
onhover
andonchecked
.I hope you find this helpful.
Happy Coding!
0 - @AdrianoEscarabotePosted about 2 years ago
Hi ArthurS, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
- To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is.
Prefer to use
rem
overpx
to have your page working better across browsers and resizing the elements properlyWe can do this to improve the design of the page:
.attribution { color: #fff; }
The rest is great!
I hope it helps... 👍
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