Feat/nombrar la sala - #33
Merged
Merged
Conversation
`noche-fiesta-79` no le recuerda a nadie que ahi esta la landing de Cinepolis, y el menu de salas se reordena por ultima visita, asi que la que buscas cambia de lugar cada vez. Ahora la sala se puede nombrar con doble clic en la cabecera. El nombre es de la SALA, no de quien lo puso: viaja en el `joined` y se propaga por socket, asi que los compas lo ven cambiar al momento y en el chat sale quien fue. Un nombre que cambia solo, sin saber por quien, se siente a error. Gana el ultimo que escribe, como el titulo de un documento compartido: es un campo chico, se toca poco, y quien no este de acuerdo lo vuelve a cambiar viendo ya lo del otro. No hace falta bloquear nada. El id NO cambia. Sigue siendo la URL y lo que se dicta por telefono, asi que renombrar no rompe ningun link ya compartido. Sin nombre se ve el id, como siempre; dejarlo vacio lo quita. El menu guarda una copia del nombre en localStorage para pintar la lista sin preguntarle al server por cada sala. Si alguien lo cambio mientras no estabas, lo ves al volver a entrar: para reconocer tus salas, alcanza. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Dos cosas que salieron probando el renombrado: El doble clic no lo descubre nadie: un titulo se ve igual sea editable o no. Ahora es un clic, y al pasar el mouse aparece una linea debajo que dice que se puede tocar. De paso es un <button>, asi que tambien se llega con Tab. Y el preview salia roto de vez en cuando con "socket hang up" en /@vite/client: Vite acepta conexiones un instante antes de poder servir de verdad, asi que la primera peticion del navegador se topaba con la conexion cortada. Recargar lo arreglaba, que era la senal de que solo faltaba esperar. Ahora el proxy reintenta con espera creciente (250ms, 500, 1s, 2s). Creciente y no fijo porque casi siempre basta el primero: la ventana mala es de decimas. Los siguientes cubren arranques lentos, que los hay (un proyecto tardo 18s en levantar). Y con techo de ~3.75s: el dev server esta en la misma maquina, asi que si a esas alturas no contesta es que esta caido, no ocupado, y seguir insistiendo solo dejaria conexiones colgadas. Solo con ECONNRESET, solo si no se le mando nada al cliente todavia, y solo en GET: el cuerpo de la peticion original ya se consumio y no se puede reenviar. Los assets del arranque, que es lo que falla, son todos GET. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
El menu se ordenaba por ultima visita, asi que cada vez que entrabas a una sala esta saltaba al principio y las demas se recorrian. La lista se revolvia sola y encontrar la de siempre era buscar donde habia quedado. Ahora es una cola: la sala nueva entra arriba y las que ya estaban conservan su sitio pase lo que pase. Se sigue anotando cuando entraste (la lista lo muestra), pero eso ya no mueve nada. Las listas guardadas de antes conservan el orden que tenian: no es exactamente el de creacion, pero de aqui en adelante deja de moverse. 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.