AM272: Nuevo botón para ver imagen, lógica para open window en android

[Pela2silveira]

Requerimiento

User Story

Funcionalidad desarrollada

Funcionalidad implementada para Android. En la sección Historia de Salud, en el detalle de categorías, ante búsqueda por prestación:

1. Si la prestación tiene metadata con studyUid (esto infiere que tiene un PACS asociado con un estudio), muestra un botón para ver la imágen
2. Cuando se apreta dicho botón abre un navegador con el estudio.

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

[negro89]

El codigo está ok, lo que no debería subir es el ionic.config. Habría que sacarlo del commit para poder mergear 👌🏼

[Pela2silveira]

revert ionic.config

[Fabio-Ramirez]

Pablo, la funcionalidad estaría correcta. Quedaría darle un toque al diseño del botón, como por ejemplo que se distinga del botón de descarga. Al igual que un "sombreadito" o algo parecido cuando se clickea. Son observaciones que pueden ser clarificadas en conjunto con la parte de UI/UX.

[Pela2silveira]

Buenas @Fabio-Ramirez, muchas gracias por los comentarios:

1. en cuanto al diseño del botón, lo tome de la librería propia del framework, no hay muchas opciones. Hay uno de un "ojo" que propuso @AgosLizzi que me parece bien.
2. en relación a sombreados y/o animaciones al momento del click estoy de acuerdo, entiendo que es algo que lo van a tener que implementar para todos los botones de la app

Quedo atento a cómo sigue el tema. Entiendo que el 2. no es limitante para el merge.

Muchas gracias por el apoyo. Saludos

[AgosLizzi]

Pablo, con respecto al icono en si, me parece que tanto la imagen como el ojo funcionan. Igualmente me inclino hacia el icono de la imagen, ya que con el otro (descarga) comparten anatomía, como sugerencia el de descarga le sacaría ese pleno celeste y lo dejaria tal cual lo muestra la interfaz de ionic (Solo contorno). También ajustaria el tamaño de ambos iconos, esta enorme... El resto de los detalles no genera conflictos para el merge.
Dejo aclarado que en sprint anteriores se estuvo trabajando sobre algunos lineamientos para al app mobile, estos incluyen un mini rediseño sobre cada pantalla, mas que nada para orderar y jerarquizar. Por lo tanto, esta pantalla se podría tener en cuenta, te dejo ejemplo:

[Pela2silveira]

@AgosLizzi mando un nuevo commit, así quedaría: