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 - Administración - Altura de IFrame no se recogía al cambiarla #502

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

ainaraRT
Copy link
Collaborator

@ainaraRT ainaraRT commented Dec 3, 2024

Description

Como se describe en el issue, no es posible establecer la altura de un campo iframe. Aunque se configure en el estudio, al insertarlo en una vista de detalle de cualquier módulo, la altura no se respeta.
Esto es debido a que en Estudio, al cambiar la altura, no se respeta el nuevo valor que se le establecía, sino que volvía a guardarse con el valor original.
Como tal, el rango de la altura es de 100 a 1024, por lo que menor o mayor a este rango no se cambiará.

Motivation and Context

Se ha añadido una condición en el que al generar la caché, se guarde el valor añadido en el párametro de la altura:
$row["ext4"] = $saveCache->ext4;
Además de añadirlo en la tabla de "módulo_cstm":
$to_save['ext4'] = (isset($field->ext4) ? $field->ext4 : '');

How To Test This

  1. Crear un campo, en cualquier módulo, de tipo iframe añadiendo una altura inicial de entre 100 y 1024.
  2. Agregar dicho campo a la vista de detalle.
  3. Añadir un iframe con dicha altura (ej: https://placehold.co/300), volver a cargar la página y comprobar que es la atura definida.
  4. Volver a estudio y editar la altura.
  5. Comprobar que se ha guardado correctamente, dentro de estudio, y comprobarlo en la vista de detalle

@ainaraRT ainaraRT self-assigned this Dec 3, 2024
Copy link

github-actions bot commented Dec 3, 2024

Actions executed at: 2024-12-19 16:33:39.

Copy link
Collaborator

@PaulaaSTIC PaulaaSTIC left a comment

Choose a reason for hiding this comment

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

No se recoge bien la altura del iframe cuando es 100, no se guarda en estudio.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - General - No se respeta la altura definida en campo iframe en vistas de detalle
2 participants