DEV Community

Discussion on: Checkmark: a beautiful to-do with Materialize

Collapse
 
gablaroche profile image
Gabriel Laroche • Edited

It's pretty nice, but I have a few suggestions:

  • The user shouldn't be able to create empty todos,
  • The site is not very accessible, be sure add focus states on interactive elements
  • Some of the icons/buttons/links don't really state their intent, I would either add a label or at the very least a title, so the user knows what the buttons do.

EDIT: I also noticed that you can add the same item over and over and that html isn't escaped, this could lead to cross site scripting

Other than that, Good job, keep it up!

Collapse
 
micahlt profile image
Micah Lindley

I created this in about 45 minutes, so it’s a work in progress. But thanks for the suggestions! I’ll definitely try implementing this stuff.