Design comparison
Solution retrospective
Changed the background color for contrast.
Community feedback
- @RSnow20Posted almost 2 years ago
Interesting idea to use "display: none" to hide the second "thank you" state of the contents. This makes it very easy to modify the contents in the future.
Also, I like the contrasting background color change. It adds a lot.
One note, you should have a check to make sure the user has a rating button selected during the submission process. If no rating is selected the user gets an error, because no rating was selected, and the ".customer-selection" span is empty.
Good work!
Marked as helpful0@estebanp2022Posted almost 2 years ago@RSnow20 thanks for calling that out - will definitely look into adding the functionality of preventing submission without a rating first!
0
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