So lässt sich das Problem reproduzieren:
Ich vermute, dass das Problem durch diesen Code verursacht wird.
Kannst du das hier auf Meta oder auf try.discourse.org nachvollziehen?
Ich kann es auf Feverbee nachvollziehen, aber hier nicht. Es könnte an den von ihnen vorgenommenen Anpassungen liegen, wenn wir es hier nicht nachvollziehen können.
Du kannst es hier nachstellen:
(Verringere die Breite, aber nicht zu sehr.)
Das ist tatsächlich ein Fehler… wenn es nur zwei Beiträge im Thema gibt und man den Bildschirm verkleinert, tritt er auf.
Ich stimme zu, dass der verlinkte Code wahrscheinlich der Übeltäter ist.
Der Link sollte stabil bleiben, da dies Teil der vorgegebenen Daten ist.
Könntest du dir das hier mal ansehen, @j.jaffeux?
Also ja, dieser Code ist dafür verantwortlich, und das passiert bei einem bestimmten Verhältnis von Fensterhöhe zu Themenhöhe.
Aber was ich nicht verstehe, ist: Was ist das erwartete Verhalten? Ich könnte diese Prüfung einfach entfernen, und es würde funktionieren, aber ich vermute, wir haben sie aus einem Grund hinzugefügt?
@eviltrout Du hast diese ursprüngliche Funktion erstellt: FIX: If a topic is really long, show the timeline · discourse/discourse@79763a6 · GitHub
Ist das nicht mehr gültig? Ist die Formel falsch? Ich denke, sie wird hier ausgelöst, obwohl das Thema eigentlich nicht so groß ist, also sollten wir das vielleicht beheben?
Außerdem habe ich festgestellt, dass alle vorgeschlagenen Themen bei dieser Berechnung der Themenhöhe verwendet wurden: ${"#topic").css("background", "red")}:
Vielleicht ist das der Grund, warum es sich seit deiner Umsetzung geändert hat.
Vielleicht sollten wir es auch eher beheben? Warum in diesem Fall den Themenfortschritt anzeigen?
Wenn es <= 3 Beiträge sind, verstecken wir den Themenfortschritt, wenn posts.height < window.height * 1,5 ist?
Ursprünglich war die Logik so gedacht, dass bei wirklich kurzen Themen kein Fortschritts-Widget angezeigt werden sollte, was meiner Meinung nach immer noch Sinn ergibt.
Ich war jedoch der Ansicht, dass der Fehler darin besteht, dass das Widget zwar angezeigt wird, aber ein Klick darauf nicht funktioniert. Eine geeignete Lösung wäre, dass ein Klick immer funktioniert, wenn das Widget angezeigt wird. Es ist für mich in Ordnung, es in anderen Fällen (bei kurzen Themen) auszublenden.
Ich frage mich, ob wir den Schwellenwert für das Ausblenden auf mindestens 1 ändern sollten – zumindest auf mobilen Geräten oder bei eingeklappten Ansichten (oder diese Logik auf mobilen Geräten bzw. bei eingeklappten Ansichten sogar ganz entfernen).
Auf mobilen Geräten hilft es bereits bei nur zwei Beiträgen sehr, zu erkennen, ob man sich im ersten oder zweiten Beitrag befindet.
Es hat einen Vorteil, die Zeitleiste auch auf dem Desktop immer anzuzeigen, z. B. wenn der Beitrag, den Sie lesen, nicht auf einen Bildschirm passt.
Das sollte es beheben:
Details im Commit.
Dieses Thema wurde automatisch 5 Tage nach der letzten Antwort geschlossen. Neue Antworten sind nicht mehr möglich.
