Repository navigation
fix(sede): los documentos se abren a la derecha y la ventana no se mueve al ampliarse - #2051
Conversation
…ana crece sin moverse La columna del consentimiento pasa a la izquierda y la ventana deja de centrarse al cambiar de tamaño, así que lo que se estaba mirando no salta. Los iconos de «Ver documentos» y «Ocultar» se intercambian para apuntar hacia donde se abre el panel.
La columna del visor de la ampliada es más estrecha que la de la ventana principal, y a lo alto la hoja se quedaba pequeña. El escritorio sigue abriendo a la página entera.
…liada Con la ventana anclada a su esquina, el visor a la izquierda empujaba la columna unos 680 px al pasar del consentimiento compacto a marcar el área.
sgomez
left a comment
There was a problem hiding this comment.
CLEAN
Revisión completa (primera revisión), contra origin/main...017aa408.
Checks todavía en curso en 017aa408: Imagen nativa, Cadena Rust, Cadena TypeScript y Analyze (rust). No los he repetido en local; el veredicto sobre ellos es de la puerta de merge.
Spec: la PR no lleva Closes #N y no he encontrado ninguna issue de origen, así que no he comprobado la fidelidad a una spec. He revisado contra lo que promete el cuerpo de la PR, y lo cumple: la columna queda a la izquierda en la ampliada y al marcar el área, fit_to deja de llamar a center(), los iconos se intercambian y ReceivedViewer abre ajustado al ancho, sin tocar el modo por defecto del escritorio.
[fix]
docs/design/ventana-de-sede.md:367-370: en «Marcar el área», la ficha sigue diciendo «visor a la izquierda y barra a la derecha». Esta PR lo contradice y además cambia esa misma geometría unas líneas más arriba (L348-352). Hay que reescribirla con la columna a la izquierda y el visor a la derecha.
[note]
- Iconos: el botón que abre el panel («Ver documentos») ahora usa
collapseSidey el que lo cierra («Ocultar») usaexpandSide. Las flechas apuntan bien, pero los nombres dicen lo contrario de lo que hace cada botón (posible Mysterious Name). Si se quiere arreglar, convendría renombrarlos según la dirección (chevronSideRight/chevronSideLefto similar) en una PR aparte. - Sin
center(), la ventana ampliada puede salirse por el borde derecho o el inferior. La PR lo asume y lo documenta. Me parece un trade-off razonable porque es lo único que X11 y Wayland hacen igual, pero conviene probarlo en una pantalla pequeña conlabvm. - El test
opens a PDF fitted to its widthsí puede fallar: conDEFAULT_ZOOM(fit-page), «Ajustar al ancho» no estaría pulsado.
| aparece ya en su posición estándar y «Continuar» está habilitado desde el | ||
| principio, porque la posición estándar ya es una respuesta válida. | ||
| - **Marcar el área tiene la geometría de la ampliada**: la columna de 400 px a | ||
| la izquierda, con cabecera y pie, y el visor a la derecha, sin frase de |
There was a problem hiding this comment.
[fix] Este cambio contradice el apartado «Marcar el área» que hay más abajo (L367-370), que sigue diciendo «visor a la izquierda y barra a la derecha». Hay que reescribirlo con la columna a la izquierda y el visor a la derecha, como aquí.
There was a problem hiding this comment.
Corregido en 583aab5: «Marcar el área» ya dice columna a la izquierda, con «En qué páginas», y visor a la derecha.
|
Corregida en 38c569c la nota de los iconos: ahora se llaman por su papel, como el resto de |
Al ver los documentos o marcar el área, la ventana de sede crecía hacia la izquierda y se recentraba, y lo que se estaba mirando saltaba de sitio. Ahora:
Cerca del borde derecho de la pantalla, la ventana ampliada puede salirse, porque ya no se recoloca.