Cómo reproducir:
Creo que el problema es causado por este código.
¿Puedes reproducirlo aquí en meta o en try.discourse.org?
Puedo reproducirlo en Feverbee, pero no aquí. Podría deberse a las personalizaciones que han implementado si no podemos reproducirlo aquí.
Puedes reproducirlo aquí:
(reduce el ancho, pero no demasiado)
Esto es, de hecho, un error… si solo hay 2 publicaciones en el tema y reduces el tamaño de la pantalla, ocurre.
Estoy de acuerdo en que es probable que el código vinculado sea el culpable.
El enlace debería permanecer estable, ya que esto forma parte de los datos iniciales.
¿Podrías echar un vistazo a este, @j.jaffeux?
Sí, este código es el responsable y ocurre en una proporción específica entre la altura de la ventana y la del tema.
Pero lo que no logro entender es: ¿cuál es el comportamiento esperado? Podría simplemente eliminar esta verificación y funcionaría, pero supongo que la añadimos por una razón.
@eviltrout, tú implementaste esta característica inicial FIX: If a topic is really long, show the timeline · discourse/discourse@79763a6 · GitHub
¿Ya no es válida? ¿La fórmula es incorrecta? Creo que se está activando aquí cuando el tema no es realmente tan grande, así que ¿deberíamos corregirlo?
Además, descubrí que todos los temas sugeridos se estaban utilizando en el cálculo de la altura del tema: ${"#topic").css("background", "red"):
Quizás esto es lo que ha cambiado desde que lo implementaste.
También, ¿quizás deberíamos solucionarlo antes? ¿Por qué mostrar el progreso del tema en este caso?
Si es <= 3 publicaciones, ¿ocultamos el progreso del tema si posts.height < window.height * 1.5?
Originalmente, la lógica pretendía no mostrar un widget de progreso si el tema es muy breve, lo cual creo que sigue teniendo sentido.
Sin embargo, tenía la impresión de que el error consistía en que el widget sí aparecía, pero al hacer clic en él no funcionaba. Una solución adecuada sería que el clic funcione siempre que se muestre. Me parece bien mantenerlo oculto en otros momentos (en temas breves).
Me pregunto si deberíamos simplemente cambiar el umbral de ocultar a 1, al menos en móviles / colapsados (o incluso eliminar esta lógica de móviles / colapsados).
En móviles, incluso si solo tienes 2 publicaciones, ayuda mucho a saber si estás en la primera o en la segunda.
Siempre hay un valor en mostrar la línea de tiempo incluso en el escritorio, por ejemplo, si la publicación que estás leyendo no cabe en una pantalla.
Eso debería solucionarlo:
Detalles en el commit.
Este tema se cerró automáticamente 5 días después de la última respuesta. Ya no se permiten nuevas respuestas.
