search

davidskalinder / mpeds-coder

MPEDS Annotation Interface
MIT License
0 stars 0 forks source link

Warn user when more than X characters are entered in a field #99

Open davidskalinder opened 4 years ago

davidskalinder commented 4 years ago

@alexhanna, it occurs to me that now that #96 has removed all character limits for input fields, there is a slight worry that somebody's cat could fall asleep on the spacebar and invisibly double our DB size with a single event description.

It seems like the best way to handle this would be a bit of JS that throws up a confirm dialog when an entry exceeds 2000 (or 1000, or whatever) characters?

Not sure how hard that would be to implement, and it's probably pretty low priority since the chances of this happening are slim (okay, very slim for a cat, but someone could start like pasting whole articles or something)... But feel free to do the thing where you say that there is some incredibly easy way to add a safeguard for this kind of thing if you know one?

alexhanna commented 4 years ago

I'm not really concerned about this but yeah good to keep it on low priority wishlist.