Submitted almost 2 years ago
Single Price Component with Dark Mode Using Redux
@JoshuaAsistio
Design comparison
SolutionDesign
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- Theme toggle is not accessible ⚠️. Use either
input="checkbox"
or twoinput="radio"
and either option should havelabel
stating what mode is active.
- I recommended looking up how to create a proper theme toggle using variables. The way your doing it, is not correct.
- The “30-day, hassle-free money back guarantee” is not a heading❌. It should instead be wrapped in a
paragraph
element.
- The button was created with the incorrect element ❌. When users click on the button they should directed to a different part of your site; the
anchor
element will allow this to happen.
More Info:📚
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 👾
Marked as helpful0 - Theme toggle is not accessible ⚠️. Use either
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