Skip to content

MISC - Implementación de nuevo menú hamburguesa - #3177

Closed
ma7payne wants to merge 1 commit into
masterfrom
MISC-410
Closed

ma7payne wants to merge 1 commit into
masterfrom
MISC-410

Conversation

@ma7payne

@ma7payne ma7payne commented Apr 24, 2025 •

Copy link
Copy Markdown
Contributor

Requerimiento

Funcionalidad desarrollada

  1. Se incorpora el uso del componente plex-menu
  2. Se configura el menu actual para adaptarse al esquema con submodulos

ℹ️ Los colores de los modulos se configuran desde el item -color- (hexa) para cada elemento de la colección "modulos".

⚠️ Es necesario tener instalada la última versión de Plex (8.0.14) y Angular v15.

El paquete de instalación tiene agregada la dependencia en este PR, pueden ejecutar el script npm install para que se descargue e instale.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@ma7payne
ma7payne requested review from a team as code owners April 24, 2025 15:07
@ma7payne
ma7payne force-pushed the MISC-410 branch 2 times, most recently from b48e00a to 61785d9 Compare April 24, 2025 18:09
@ma7payne
ma7payne force-pushed the MISC-410 branch 3 times, most recently from 1595476 to a053d94 Compare May 16, 2025 13:36
@negro89

negro89 commented May 19, 2025 •

Copy link
Copy Markdown
Contributor

Mati, quedó espectacular 👏🏼👏🏼👏🏼 la unica observación que tengo, en caso de que se pudiera modificar, es que al desplegar algunas de las opciones colapsables y luego navegar a otra opción del menú, las colapsables permanecen desplegadas.
Dejo captura obtenida luego de seleccionar algunas opciones en el siguiente orden: internación > huds > citas > salud perinatal y finalmente volver a abrir el menú.
image

Espero haberme explicado, cualquier cosa consultame 👍🏼

EDIT 21/05: Mati, al final se consensuó que el único colapsable desplegado al abrir el menú sea el que corresponde a la pantalla actual, resaltando dentro el item correspondiente a la pantalla que se esta viendo.

@Fabio-Ramirez Fabio-Ramirez added the changes requested Se solicitaron cambios label May 20, 2025

@AgosLizzi AgosLizzi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Mati, Que lindo se ve... Me sumo a la corrección que te dejo @negro89 y te dejo solo un pequeño detalle visual. El contraste que se genera entre el blanco y amarillo no es tan intenso, lo que puede provocar que el blanco en este caso no se llegue a visualizar, sumado a que la tipografía tiene un trazo fino. Probaría con otro tipo de amarillo no tan puro, tal vez el que tiene "punto inicio fichas".

@ma7payne

Copy link
Copy Markdown
Contributor Author

Hola! Ya se charlaron los cambios sugeridos por @negro89 y se van a aplicar en la próxima versión. Por otro lado, los colores que menciona @AgosLizzi se aplican desde la app asi que no habria que modificar el componente de plex. Gracias a ambos!

@negro89

negro89 commented May 22, 2025

Copy link
Copy Markdown
Contributor

Mati, edité el comentario anterior sobre los colapsables del menú. Te dejo algo sobre el pegado sin formato de plex

Realicé una nueva revisión ya que no había tenido en cuenta la incorporación del pegado sin formato en los campos de texto libre. Agrego un par de cositas a considerar:

  1. Un pequeño error que aparece al momento de registrar una prestación..
    image
    image

  2. El texto se pega sin formato (ok) pero al darle formato personalizado no se respeta la visualización en la vista de la prestacion guardada pero aun no validada. Luego al imprimir nuevamente respeta el formato correctamente. Dejo capturas:

  • Edición/registro (ok):
    image

  • Validación y post-validación (En este punto habría que ajustar):
    image
    image

  • PDF (ok):
    image

Cualquier cosa consultame 👍🏼

@ma7payne ma7payne added En pausa Desarrollo pausado and removed changes requested Se solicitaron cambios labels Jul 10, 2025
@ma7payne

Copy link
Copy Markdown
Contributor Author

Mati, edité el comentario anterior sobre los colapsables del menú. Te dejo algo sobre el pegado sin formato de plex

Realicé una nueva revisión ya que no había tenido en cuenta la incorporación del pegado sin formato en los campos de texto libre. Agrego un par de cositas a considerar:

  1. Un pequeño error que aparece al momento de registrar una prestación..
    image
    image
  2. El texto se pega sin formato (ok) pero al darle formato personalizado no se respeta la visualización en la vista de la prestacion guardada pero aun no validada. Luego al imprimir nuevamente respeta el formato correctamente. Dejo capturas:
  • Edición/registro (ok):
    image
  • Validación y post-validación (En este punto habría que ajustar):
    image
    image
  • PDF (ok):
    image

Cualquier cosa consultame 👍🏼

Ya quedó revisado lo comentado por @negro89. El funcionamiento actual es el esperado.

@ma7payne ma7payne added changes done and removed En pausa Desarrollo pausado labels Jul 14, 2025
@ma7payne ma7payne added the En pausa Desarrollo pausado label Jul 31, 2025
@silviroa

Copy link
Copy Markdown
Contributor

Se incluye el cambio en PR #3265

@silviroa silviroa closed this Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants