HTML5, CSS3, JS

Community feedback

• 

  Faris P•2,810

  @FarisPalayiPosted over 3 years ago

  Looks good and responds well 👍

  Although, there are some things to improve when it comes to accessibility.

  • Toggles should be a focusable and interactive element, currently, you are using a div to create the toggle, which is not focusable intrinsically, so I suggest you to use a checkbox or radio button as the toggle.
  • If you try to navigate through the site, there is no visual feedback on where your focus is. Try to add :focus state styles.
  • And also you can add cursor: pointer to better indicate interactivity on your 'start my trial' button.

  Here are 2 articles that I think you might find helpful :

  HTML: A good basis for accessibility, Semantic HTML5 Elements Explained

  Besides these, site is working well and have fun coding and learning ✨

  Marked as helpful

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  you should combine your two buttons into one they are currently fighting each other when the user interacts with them

  Marked as helpful

  0

  ---

  

  Dmytro•305

  @DmytroTarasiukPosted over 3 years ago

  @palgramming I've just decided two add to buttons instead of 2 as was in challenge. In my mind it's more readable. It is fighting because you can choose only one option and then according to the selected option the input range will take the right labels

  0