Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive Rate Card

@hatchcloud

Desktop design screenshot for the Interactive rating component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

T
Grace 29,930

@grace-snow

Posted

I also strongly recommend you remove and stop using the font-size: 62.5%; as soon as possible. It brings little to no benefit (you don’t need to make 1rem equal 10px) and introduces huge accessibility issues. I’ll be positing about this very soon on FEDmentor.dev if you want to get notified when the post goes live

Marked as helpful

1
T
Grace 29,930

@grace-snow

Posted

Hi

I’m afraid this is completely inaccessible. You must not use js to create html element’s unnecessarily, especially when they are not the correct html.

This must be a form with radio inputs. There is no other correct way to mark this up.

Once that’s done, the JavaScript part of this challenge is really tiny. Minimally it’s about 3-5 lines - just getting the chosen radio value from the form and showing it on the thank you.

If showing the thank you panel on the same page instead of redirecting as per a standard form submit, then it’s another couple of lines of js to show/hide the panels.

Most important of all is to refactor the html. Then refactor the js.

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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