search

concristina / carreros

Plataforma de fiscalizacion y herramientas de campaña
0 stars 4 forks source link

Multiples imagenes #43

Open dtokman opened 5 years ago

dtokman commented 5 years ago

Al cargar actas, si alguna tiene varias imágenes, al cargar datos y hacer un scroll down se terminan perdiendo.

Multiples imagenes.zip

mgaitan commented 5 years ago

que version de chrome estás usando? podes fijarte si te da algun error de javascript?

se tienen que ver como pestañas asi

image

el mio es 

Google Chrome | 71.0.3578.98 (Official Build) (64-bit)
-- | --
Revision | 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS | Linux

fijate porfa con otro browser si lo ves bien

dtokman commented 5 years ago

En lo personal, probé la app con Microsoft EdgeHTML 17.17134 Igualmente, fuimos varios los que probamos y con Fernando juntamos todos los issues y yo los cargué, por lo que debería verificar qué navegador está utilizando cada uno. Tenemos garantizado las versiones de los distintos navegadores con los que debería funcionar correctamente la aplicación?

atcach commented 5 years ago

Al menos con firefox 65.0.1 (64 bit) y con chrome Versión 55.0.2883.87 (64-bit) se ve bien. El único problema es que si son más de 4 imágenes, el scrolling de las pestañas en firefox no se puede hacer.

En chrome si, pero es medio complicado y sucede de vez en cuando. Hay que pararse sobre la pestaña con el nro. de la foto y scrollear ahí. A veces no anda y es muy poco intuitivo. Aparentemente el comportamiento deseado no es este, sino el de firefox porque scrollea todo el panel de la derecha. Adjunto imágen de firefox que al haber scrolleado se ve que scrollea solo el panel izquierdo y foto en chrome que scrolleo ambos paneles (al hacer esto se ven las otras fotos) Firefox: image

Chrome: image

Chrome que al hacer scroll aparecen las otras pestañas image

Al parecer es porque la parte esta está con position sticky que no sirve para chrome anterior a 56 (lo sacaron y vuelven a poner en esta version). Esa propiedad está en .acta

Se le puede cambiar por fixed. Eso queda un poco más feo porque el header azul se ve de fondo un poco, pero no tanto y aparentemente sería más estándar.

Además, para hacer andar todo junto, le sacaría el tamaño al tab. En realidad no al .tab, sino a este en particular

.tabs { position: relative; overflow-x: auto; overflow-y: hidden; height: 48px; .... }

Ese height 48px, al eliminarlo muestra bien. y el .tabs lo cambiaría por algo específico para no afectar a todo el contenido. Algo del estilo .right-panel .acta .card-content .tabs Actualmente está en materialize.frontend.min en la línea 2. Pero no se como hacer un override.