All comments

• 

  Ndukwe Ihuoma•130

  @Ihuoma5

  Submitted about 3 years ago

  

  To-do app

  1

  40
  

  Matt Mackenzie•80

  @Mmackz

  Posted about 3 years ago

  The use of the button element without having any text is causing accessibility issues. Instead of using buttons for the checkbox element, you should consider using an input with the type attribute set to checkbox.

  While this is a little hard to style, there are plenty of good tutorials that show how this can be done.

  https://www.w3schools.com/howto/howto_css_custom_checkbox.asp

  Also, I noticed a few bugs, such as only being able to check the 2nd todo from the top, and the inability to uncheck any of the todos.

  0

  Marked as helpful