-
Notifications
You must be signed in to change notification settings - Fork 5
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
RUP - Implementa mockups en punto de inicio #2945
Conversation
3671d4c
to
41f0c3b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mati, te dejo un par de cositas que vi:
-
Los botones cambian de visual cuando cambio de pantalla. Fijate también que en el mismo video cuando quiero abrir un listado "fuera de agenda" no me deja, recien cuando lo abrí desde otro dif, me deja pasar al que quiero. te dejo video: https://www.loom.com/share/1e8b2faa976a4e858552ca29969a52ec +
-
Re ordenar tooltip hacia la izquierda, no importa que se superponga con los botones.
-
No carga bien las agendas, por alguna razón cuando cambio de pantalla, me refresca y ahi carga las que corresponde. Te dejo video: https://www.loom.com/share/683ed0805b064cae81c68194be23c2f0
431db80
to
e724e4b
Compare
Listos los cambios! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mati, van correcciones:
- Hay varios badge que estan mal acomodados o quedaron fuera de contextos. Ye adjunto capturas:
Si vas al mockup los badge estan en otra posición, no se si fue adrede cambiarlos o simplemente quedaron así
- Te adjunto un video donde sucede algo parecido a la primera observación, pero con el botón, cuando voy a "registrar inasistencia", el botón de "revertir cambios" quedá en un lugar donde no debería. https://www.loom.com/share/c31658f8f48e4b05bbf07f4601be6603
- Este tooltip, que paso? Observar también el botón, me parece que no es correcto.
- Cuando doy un sobreturno, queda así. Acomodar situación con botones y agregar plex title de sobreturnos, por favor.
- La acción de autocitado esta mal. Fijate que en demo esta bien la implementación... Como primera instancia en este caso el botón de "iniciar prestación" esta de más.
52478e8
to
997a3ec
Compare
@AgosLizzi Estan listos los nuevos cambios! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mati, van nuevos cambios:
- Hay un problema con el scroll y el label. Desde mi opinión, no se si es necesario el doble scroll, el que va sobre el listado ocasiona muchos problemas. Y luego el label queda como estático. Va video adjunto https://www.loom.com/share/0daf14ccbcb84e098dc11c606a265120
- En sobreturno los badge se repiten y el botón queda mal ubicado
- Consulta, hay posibilidad que los badge queden uno al lado del otro y no uno sobre el otro?
- Y por último, muy parecido al ejemplo 1, desde autocitado sucede lo siguiente:
https://www.loom.com/share/2d0c44a14e3f47c9a25e55dd84014e55
*El principio del video, yo me confundí y empecé a grabar en demo, desestimalo por fa.
✅ TEST OK (#11041) |
Agregar tooltip faltante para cerrar el sidebar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mati se ve hermosa la nueva pantalla :D
Te comento por aca lo que estuvimos charlando que falta revisar:
No se visualizan los fuera de agenda algunos dias (parece ser que es cuando no hay ninguna agenda para visualizar).
Cuando se visualizan los fuera de agenda debería ser un sólo item para todos los fuera de agenda cargados en el día, en cambio ahora se visualiza uno por cada prestacion cargada.
Dato de color aprovecharía a sacar el campo requerido en el selec de prestacion, ya que nunca se aplica como tal
130fc21
to
1a45df1
Compare
@MCele Listos los últimos cambios en la visualización del listado de agendas. |
421e9da
to
83b7469
Compare
Test OK #11094 |
@negro89 Listos los cambios en los botones y en la actualización de la vista para evitar el cambio de agenda (puntos 2 y 3). El punto numero 1 corresponde a una vista del gestor de agendas. Quedo atento! |
Requerimiento
https://proyectos.andes.gob.ar/browse/RUP-370
Funcionalidad desarrollada
UserStory llegó a completarse
Requiere actualizaciones en la base de datos
Requiere actualizaciones en la API
Requiere actualizaciones en andes-test-integracion