Feat/la sala en el celular - #41
Merged
Merged
Conversation
Estaba hecha para pantalla ancha: el chat a la izquierda y el preview a la derecha, los dos a la vez. En un telefono eso deja las dos cosas inservibles, y ya habia costado una demo (hubo que llevar la laptop en vez de enseñarlo en el celular). En pantallas de menos de 820px el chat pasa a ser una pestaña mas, junto a "La app" y "El back", y se ve una a la vez. Esa pestaña no existe en escritorio, donde el chat vive en su columna y no es una vista. La cabecera se queda siempre visible aunque el chat no lo este: vive dentro de el, y esconderlo se llevaba el nombre de la sala, el menu y el boton de crear. En vez de moverla en el JSX (que cambiaria el escritorio, donde esta bien), el chat se encoge a solo su cabecera cuando la pestaña activa es otra. Tambien: la barra de arriba baja de renglon en vez de salirse de pantalla, los paneles flotantes se ajustan al ancho disponible (median mas que un telefono), y las alturas usan `dvh` ademas de `vh`, porque `vh` ignora la barra del navegador del celular y cortaba justo la caja de escribir. El escritorio no cambia: todo vive dentro de la media query salvo la linea que esconde ahi la pestaña nueva. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
No queda bien todavia, pero deja de ser inusable: antes el chat y el preview se repartian una pantalla de telefono y las dos cosas quedaban ilegibles. Los arreglos: la barra de arriba deja de salirse por la izquierda (los botones no tenian por que mantener su ancho, y empujaban la fila entera fuera de pantalla); las pestañas se anclan abajo como barra de navegacion, y la sala se acorta para hacerles hueco en vez de crecer y que el `overflow: hidden` del body se las coma; y el preview se contiene, porque la app de la sala puede ser mas ancha que el telefono y arrastraba todo de lado. De paso, sin sala abierta ya no se pide `/rooms/null/export/estado`: salia un 404 en cada visita a la Sala vacia desde que se quito la portada. El escritorio no cambia: todo vive dentro de la media query salvo la linea que esconde ahi la pestaña de chat. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.