noi-techpark / it.bz.opendatahub.epaper.webapp

0 stars 5 forks source link

Error messages are too general, if an error occurs the user should be notified with a message that allows him to understand where the problem is. #68

Open sseppi opened 2 years ago

sseppi commented 2 years ago

For example if I try to save a template without loading an image I get the generic message "Faild to save template". In this case it is difficult for the user to understand how to solve the problem. We think that in general the messages to the users should help them to solve the problem. In this specific case it could be "Error: please add an image" and, if possible, the empty or wrong field should be marked in red.

SaimonasFOS commented 2 years ago

@Piiit @sseppi

I added field validation in the webapp but I kept the default message. In the last meeting you requested that error messages should be more visible/more prominent. That is not an easy thing to do - the current message is the default browser message and it's style cannot be changed. That means I have to write my own form validation logic and show custom errors which is not trivial, especially in case of dynamic list like with image fields list. Besides, the message style depends on the browser too, on Chrome/Edge it's more visible, I demonstrated it on Firefox.

Piiit commented 2 years ago

@sseppi @SaimonasFOS OK, I suggest to keep it as is, but move this issue to "To Do" and postpone it to MVP2...