@awexli
Posted
Hey Shawn, nice solution :-)
Instead of using span
for the titles "How did we do?" and "Thank you!", it might be semantically better to use heading tags (<h1 />
)
Beyond the code itself, one thing to think about is the user experience. For example, what should happen if a user clicks submit without selecting any options? Should there be some type of UI feedback letting the user know they must select at least one option? How would you represent that? Maybe a little message below? It's good to keep user experience in mind as a Frontend dev -- good luck on your journey!
Marked as helpful
@OGShawnLee
Posted
@awexli Hello Alex. Yes, I should have used headings and about the UX I actually thought of adding a third state that handled errors but I didn't add it at the end. I will commit those fixes soon. Thanks for replying! :)