@elaineleung
Posted
Hi Joel, well done! Everything's working as it should, and I like how you included some logic for handling what happens when the user clicks submit without clicking a score. Great job also in getting it to look really close to the design. I haven't used jQuery in a really long time, so I'm out of touch with it; I did notice that it took a while to load the component, and so that might be due to the use of jQuery. I see that you had placed the script right under the body tag, which is a very interesting choice. I'm not familiar enough with it, but I would think it should go before the closing body tag instead of right after the opening body tag. Anyway, that's my only comment on jQuery.
The only thing left now is to check out the accessibility and HTML issues in the report; you're missing a main tag, but that's fairly easy to fix. All you need to do is change <div class="card"> to <main class="card">, or alternatively, you can just add a new main tag as a container, which is what I would do in this situation.
Hope some of this helps you out!
Marked as helpful
