@ApplePieGiraffe
Posted
Hi, Grant Ramsay! 👋
Great job on this challenge! 👏 Your solution looks good and works well! 🙌
A few things I'd like to suggest are,
- Using a
form
element in your solution, which will make the component much more accessible and make it easier to implement the interactive section of the card. Use radio elements for the for the rating buttons (which will take care of only one rating being selected at a time), and make sure the submit button is inside the form so that it will submit the form when clicked. You can then select the value of the radio elements in your JS and use that to set the next view of the component. - Using less
div
s for everything in your markup and more semantic HTML elements (such asp
or heading tags). Learn more about semantic HTML and why it's important here. 😉
Hope you find this helpful. 😊
Keep coding (and happy coding, too)! 😁
Marked as helpful
@seapagan
Posted
@ApplePieGiraffe
Thank you for the brilliant and helpful feedback! 😊
Great idea on using radio buttons thanks, I'll definitely try that and convert it to a form, and see how it comes out. Would definitely make the JS easier. 😄
Semantic HTML, yeah I know this is one of my weaknesses 😂. I'm working to improve that in future challenges, so thanks for the link, I'll defo work through that.
Have a great evening! 👍️
@ApplePieGiraffe
Posted
@seapagan
Glad you found those points helpful! Have a great day, too! ☀️