Línea de tiempo superpuesta "Temas sugeridos"

Hola.
En los temas cortos, la línea de tiempo se superpone a la sección de “Temas sugeridos”:


Creo que la línea de tiempo podría ocultarse en los temas sin respuestas, o la altura mínima del contenedor de publicaciones debería ser aproximadamente la misma que la de la línea de tiempo.

4 Me gusta

Por lo que recuerdo, esto ha estado presente desde hace un tiempo y es propio de pantallas pequeñas y temas de una sola línea con un solo mensaje, pero ¿quizás hemos retrocedido? :thinking:

Esto también ocurre cuando el contenido es lo suficientemente largo y hay algo de contenido encima de la publicación

Definitivamente veo esto.

@awesomerobot me recordó que solíamos ocultar la línea de tiempo para temas de un solo post, ¿verdad? Quizás por eso lo vemos más a menudo, o quizás es una regresión real. De todos modos, en este caso, todos los elementos de la línea de tiempo están duplicados en los controles en la parte inferior del primer (y único) post…

1 me gusta

Sí, creo que lo hemos estado ocultando desde 2016: UX: Hide all timeline controls when fewer than 3 posts · discourse/discourse@b9f8264 · GitHub

Hasta hace poco tiempo, la línea de tiempo ocultaba

@j.jaffeux, ¿podría ser que tu cambio reciente haya causado este retroceso?

3 Me gusta

Definitivamente deberíamos ocultar la línea de tiempo cuando solo haya UNA publicación. Tres parece un poco excesivo. Pero una…

1 me gusta

El gran problema con una sola publicación es que, cuando resulta realmente útil como esta, no funciona correctamente. No soportamos la navegación a mitad de una publicación enorme, por lo que se vuelve muy, muy, muy, muy confuso. Mucho peor que el problema de renderizado.

2 Me gusta

Hágalo según la longitud del post más un solo post, entonces… ¿no necesitamos una línea de tiempo vertical para un solo post de una oración en un solo tema, verdad?

Absolutamente no, pero necesitamos arreglar la implementación; creo que eso es lo que estoy tratando de decir.

Si tuviera que elegir entre eliminarlo incondicionalmente o conservarlo para publicaciones individuales excepcionalmente largas, elegiría eliminarlo incondicionalmente.

Sin embargo, si arreglamos la implementación subyacente, apoyaría totalmente conservarlo para la increíble guía de @Johani; sería de gran ayuda.

La lógica debería ser

si hay un solo mensaje en el tema Y si ese mensaje tiene menos de .. 1000* caracteres .. suprimir completamente la línea de tiempo

Incluso dos mensajes de una sola oración probablemente sean seguros, ¿verdad?

  • pero ¿qué pasa si todos son caracteres individuales y CR/LF, etc.? No lo sé. Creo que una verificación aproximada está bien; no necesitamos preocuparnos por casos extremos insensatos aquí.
1 me gusta

Claro, podemos hacer algo aquí incluso solo con CSS. Pero tenemos que arreglar la barra de desplazamiento si optamos por este enfoque.

De lo contrario, también podríamos simplemente cambiarlo a if 1 post then hide timeline, que es aún más sencillo.

A menos que esa única publicación sea una monstruosidad de 30.000 caracteres, no queremos que esté oculta, así que… no.

Sí, queremos que esté oculto… lo siento… ese es mi punto. A menos que solucionemos este problema:

Arrastro la línea de tiempo hacia abajo:

Sigue en la parte superior. :crying_cat_face:

Esta barra de desplazamiento, sin embargo, funciona como se espera si uso la nativa del navegador:

Si la dejamos así para temas con un solo post gigante, simplemente no funciona correctamente, lo cual es muy confuso. Así que tenemos que solucionarlo.

1 me gusta

Vale, necesitamos buenos casos de prueba informales para asegurarnos de haber cubierto los que nos importan.

1 me gusta

No, este no es el commit.

El commit relacionado es muy probablemente:

Así que antes de este commit, la lógica estaba de alguna manera duplicada y diferente, y también estaba causando un widget de timeline-controls vacío (pero visible).

Lo revisaré hoy.

3 Me gusta

Mi intento de solucionar el solapamiento de la línea de tiempo está aquí:

No soluciona lo que explicaste, que al hacer clic o arrastrar en la línea de tiempo no cambia la posición de desplazamiento (scrollPosition) de la ventana. Lo revisé rápidamente y creo que esto debería ser un proyecto dedicado; siempre ha sido así y cambiarlo tendría MUCHAS implicaciones.

Si solo hay una publicación, ahora verificaré su altura y ocultaré el controlador de desplazamiento si es menor a 1000 px.

Representa una publicación de aproximadamente esta longitud:

Ejemplo con una publicación corta:

Este commit también corrige otro error que hacía que el botón de administrador del tema apareciera en una posición incorrecta al recargar la página completa:

Así:

En lugar de esta posición después de la corrección:

También verifiqué que dos publicaciones con la cantidad mínima de contenido no se solapen:

Nada debería haber cambiado en dispositivos móviles.

4 Me gusta

¡Excelente, gracias, Joffrey :smiling_cat_with_heart_eyes:!

2 Me gusta

Esto ya está fusionado; se puede ver en acción en este tema, por ejemplo (al menos mientras solo haya un mensaje):

3 Me gusta

Este tema se cerró automáticamente después de 5 días. Ya no se permiten nuevas respuestas.