Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix - Casos - Mostrar editor texto enriquecido en el campo Resolución #390

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ManuSinergiaCRM
Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM commented Sep 16, 2024

Descripción

Al igual que ocurre con el campo Descripción, este PR añade la propiedad editor = 'html' en la definición del campo Resolución.

Pruebas

  1. Activar el módulo de Casos y acceder a la vista de creación o de edición de un registro
  2. Comprobar que el campo Resolución se muestra con el editor de texto enriquecido

@ManuSinergiaCRM ManuSinergiaCRM added the bug Something isn't working label Sep 16, 2024
@ManuSinergiaCRM ManuSinergiaCRM self-assigned this Sep 16, 2024
Copy link

Actions executed at: 2024-09-16 12:46:18.

Copy link
Collaborator

@ainaraRT ainaraRT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(A)Probado

Copy link
Collaborator

@juanSTIC juanSTIC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lo que se añade en este PR funciona correctamente, pero pienso que si se habilita un campo HTML debería habilitarse también en la edición inline ¿cómo lo ves @ManuSinergiaCRM?

@ManuSinergiaCRM
Copy link
Collaborator Author

Lo que se añade en este PR funciona correctamente, pero pienso que si se habilita un campo HTML debería habilitarse también en la edición inline ¿cómo lo ves @ManuSinergiaCRM?

No es exactamente habilitar un campo HTML, si no que esa propiedad hace que a un campo de tipo texto se le cargue el editor enriquecido al acceder a la vista de edición.

Por coherencia, de acuerdo con Juan en que el editor enriquecido debería ser mostrado también en la edición inline. Y también, que acabo de comprobar que tampoco se muestra, en el formulario de creación rápida.

Por último, me fijé en el campo Descripción (de tipo texto) para que el campo Resolución se comportará igual, y así lo hace ya que para el campo Descripción tampoco se muestra el editor ni en la creación rápida ni en la edición inline.

En consecuencia, propondría:

  1. Dejarlo como está ya que es un módulo que por defecto no está activado.
  2. Investigar que ocurre por detrás al activar esa propiedad en un campo y hacer que el editor se muestre en las diferentes vistas de edición (Habría que probar tb la vista de popup)

Sea cual sea la opción, le crearía un Issue o un PR a SA.

@AlbertoSTIC , ¿tú qué piensas?

@AlbertoSTIC
Copy link
Collaborator

Lo que se añade en este PR funciona correctamente, pero pienso que si se habilita un campo HTML debería habilitarse también en la edición inline ¿cómo lo ves @ManuSinergiaCRM?

No es exactamente habilitar un campo HTML, si no que esa propiedad hace que a un campo de tipo texto se le cargue el editor enriquecido al acceder a la vista de edición.

Por coherencia, de acuerdo con Juan en que el editor enriquecido debería ser mostrado también en la edición inline. Y también, que acabo de comprobar que tampoco se muestra, en el formulario de creación rápida.

Por último, me fijé en el campo Descripción (de tipo texto) para que el campo Resolución se comportará igual, y así lo hace ya que para el campo Descripción tampoco se muestra el editor ni en la creación rápida ni en la edición inline.

En consecuencia, propondría:

  1. Dejarlo como está ya que es un módulo que por defecto no está activado.
  2. Investigar que ocurre por detrás al activar esa propiedad en un campo y hacer que el editor se muestre en las diferentes vistas de edición (Habría que probar tb la vista de popup)

Sea cual sea la opción, le crearía un Issue o un PR a SA.

@AlbertoSTIC , ¿tú qué piensas?

Por ahora dejaría este PR tal y como está y abriría Issue con lo que comenta Juan.

@ManuSinergiaCRM
Copy link
Collaborator Author

Lo que se añade en este PR funciona correctamente, pero pienso que si se habilita un campo HTML debería habilitarse también en la edición inline ¿cómo lo ves @ManuSinergiaCRM?

No es exactamente habilitar un campo HTML, si no que esa propiedad hace que a un campo de tipo texto se le cargue el editor enriquecido al acceder a la vista de edición.
Por coherencia, de acuerdo con Juan en que el editor enriquecido debería ser mostrado también en la edición inline. Y también, que acabo de comprobar que tampoco se muestra, en el formulario de creación rápida.
Por último, me fijé en el campo Descripción (de tipo texto) para que el campo Resolución se comportará igual, y así lo hace ya que para el campo Descripción tampoco se muestra el editor ni en la creación rápida ni en la edición inline.
En consecuencia, propondría:

  1. Dejarlo como está ya que es un módulo que por defecto no está activado.
  2. Investigar que ocurre por detrás al activar esa propiedad en un campo y hacer que el editor se muestre en las diferentes vistas de edición (Habría que probar tb la vista de popup)

Sea cual sea la opción, le crearía un Issue o un PR a SA.
@AlbertoSTIC , ¿tú qué piensas?

Por ahora dejaría este PR tal y como está y abriría Issue con lo que comenta Juan.

Creado el issue: #396

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Casos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - Casos - El editor texto enriquecido no se muestra en el campo Resolución
4 participants