Responsive Rate Card

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted almost 2 years ago

  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•31,270

  @grace-snowPosted almost 2 years ago

  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