Skip to content

Fix: avisa cuando se intenta guardar un comentario vacío en una instancia

belen.varsi requested to merge fix/mensaje-comentario-vacio-instancia into dev

Mantis 0044469

Qué

Corrige el Mantis 0044469: en el detalle de una instancia, si el comentario estaba vacío o tenía solo espacios, el botón "Comentar" quedaba deshabilitado y no se mostraba ningún mensaje que explicara por qué.

Por qué

Un botón deshabilitado sin explicación no le dice al usuario qué tiene que corregir. El caso más confuso es el de solo espacios: el campo parece tener contenido y aun así no se puede comentar.

Cambios

src/components/instancias/InstanciaComentarios.tsx:

  • El botón "Comentar" ya no se deshabilita por contenido vacío. Al presionarlo con el campo vacío o con solo espacios, no se guarda nada y aparece debajo del campo el mensaje "El comentario no puede estar vacío.", con el borde del campo en rojo.
  • El mensaje desaparece al empezar a escribir.
  • Mismo criterio al editar un comentario: si se borra todo el texto y se presiona "Guardar", aparece el mismo mensaje. "Guardar" sigue deshabilitado mientras el texto no cambie.
  • El mensaje queda asociado al campo con aria-invalid y aria-describedby.

La validación es en pantalla: no se hace ninguna llamada al backend con el comentario vacío.

Aplica también a los comentarios confidenciales y al detalle de una Incidencia, que usan el mismo componente.

Cómo probar

  1. Abrir el detalle de una instancia activa con un usuario que pueda comentar.
  2. En "Comentarios", dejar el campo vacío y presionar "Comentar": no se guarda y aparece "El comentario no puede estar vacío."
  3. Escribir solo espacios y presionar "Comentar": mismo resultado.
  4. Escribir un texto: el mensaje desaparece. Presionar "Comentar": el comentario se guarda.
  5. Editar un comentario, borrar todo el texto y presionar "Guardar": aparece el mismo mensaje y el comentario no cambia.
  6. Repetir el paso 2 en "Comentarios confidenciales" y en el detalle de una Incidencia.

Cambios en BD

Ninguno.

Edited by belen.varsi

Merge request reports

Loading