Ted Rinquest• 440
@climb512
Posted
Looks nice!
I can offer a bit of my first impressions after glancing through the code. You may want to add descriptive class names, which makes any code review or later revisions much easier (Semantic CSS). Also, you may want to add .m::focus {} with the same css as .m::hover {} to keep the active state formatting on the button when clicked.
When I did this challenge I went ahead and learned how to add the data in data.json using plain JS (no libraries). There are a million resources for this, but I used https://www.ajaypalcheema.com/vanilla-js-dom/ and it helped me a lot. It could be a great opportunity to learn a library like jQuery too if that's what you wanted.
Keep coding!
Marked as helpful
1
