fm-accessability-rating-comp

Community feedback

• 

  EL harabiy•250

  @Mubarak-AdeyemiPosted about 2 months ago

  1. JavaScript Refinement: Rating Selection Validation: Ensure users can't submit without selecting a rating. You can add a simple validation to check if any radio button is selected before showing the success message.

  2. CSS Refinement: Radio Button Accessibility: The input should have better focus states for keyboard accessibility. Consider adding a :focus style to labels for users navigating via the keyboard.

  3. Simplifying the Success Message:

  • Instead of hiding and showing sections with style.display, consider using classList.toggle to add a class like hidden to simplify the logic and maintain better readability in your JavaScript.

  4. File Separation:

  • While embedding styles and scripts in the HTML works for small projects, separating CSS and JS into their own files is good for scalability and maintainability.

  Marked as helpful

  0