Submitted over 3 years ago
Pricing Toggle with HTML, SCSS, JS, and Mobile First.
@Lusk1nha
Design comparison
SolutionDesign
Solution retrospective
This is my first project using SASS.
So, if someone can take a look at the code and give some feedback I would be very grateful.😃
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hi, Lucas Pedro! 👋
Well done on this challenge! 👍 Your solution looks good and is responsive and the toggle-switch works well! 🙌 Kudos for trying out Sass, too, BTW! 👏
I'd like to suggest,
- Adding a heading to each of the pricing cards to identify them (since you've chose to use the
<section>
element for each of the cards) and clear up those errors on your solution report. - Making the toggle-switch tabbable so that keyboard users can access and use the toggle-switch.
You could consider splitting your Sass code into separate files a little more (like for the pricing cards, perhaps) but given that this project isn't so big, it might not matter too much. Maybe in future, larger projects! 😉
Keep coding (and happy coding, too)! 😁
1 - Adding a heading to each of the pricing cards to identify them (since you've chose to use the
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