Zeitachne überlappt „Vorgeschlagene Themen"

Hallo.
Bei kurzen Themen überschneidet sich die Zeitleiste mit dem Abschnitt „Empfohlene Themen":


Ich vermute, die Zeitleiste könnte bei Themen ohne Antworten ausgeblendet werden, oder die Mindesthöhe des Beitragscontainers sollte etwa der der Zeitleiste entsprechen?

4 „Gefällt mir“

Soweit ich mich erinnere, gibt es das schon eine Weile und es ist typisch für kurze Bildschirme und Einzeiler-Themen mit nur einem Beitrag, aber vielleicht haben wir uns zurückentwickelt? :thinking:

Das tritt auch auf, wenn der Inhalt lang genug ist und sich über dem Beitrag noch weiterer Inhalt befindet.

Das sehe ich definitiv.

@awesomerobot hat mich daran erinnert, dass wir die Timeline für Themen mit nur einem Beitrag früher unterdrückt haben, oder? Vielleicht sehen wir sie deshalb häufiger, oder ist es ein tatsächlicher Fehler? In diesem Fall sind ohnehin alle Timeline-Elemente in den Steuerelementen am Ende des ersten (und einzigen) Beitrags dupliziert…

1 „Gefällt mir“

Ja, ich denke, wir haben es seit 2016 versteckt: UX: Hide all timeline controls when fewer than 3 posts · discourse/discourse@b9f8264 · GitHub

Bis vor kurzem wurde die Zeitleiste ausgeblendet

@j.jaffeux, könnte deine kürzliche Änderung dazu geführt haben, dass dies wieder aufgetreten ist?

3 „Gefällt mir“

Wir sollten die Zeitleiste definitiv ausblenden, wenn es nur EINEN Beitrag gibt. Drei wirken schon etwas übertrieben. Aber einer…

1 „Gefällt mir“

Das große Problem bei einem einzigen Post ist, dass er, wenn er tatsächlich nützlich ist wie dieser hier, nicht richtig funktioniert. Wir unterstützen keine Navigation mitten in einem riesigen Post, was es sehr, sehr, sehr, sehr verwirrend macht. Viel schlimmer als nur das Rendering-Problem.

2 „Gefällt mir“

Machen wir es basierend auf der Beitragslänge plus einem einzelnen Beitrag. Wir brauchen doch keine vertikale Zeitachse für einen einzeiligen Beitrag in einem einzigen Thema, oder?

Absolut nicht, aber wir müssen die Implementierung beheben. Ich glaube, darauf will ich hinaus.

Wenn ich mich zwischen dem bedingungslosen Entfernen oder dem Beibehalten für sehr lange Einzelposts als Ausreißer entscheiden müsste, würde ich das bedingungslose Entfernen wählen.

Wenn wir jedoch die zugrunde liegende Implementierung beheben, würde ich das Beibehalten für @Johanis erstaunlichen Guide absolut unterstützen. Das wäre super hilfreich.

Die Logik sollte so aussehen:

Wenn ein einzelner Beitrag im Thema vorhanden ist UND dieser Beitrag weniger als .. 1000* Zeichen hat .. dann die Timeline vollständig unterdrücken.

Selbst zwei Beiträge mit je einem Satz sollten wahrscheinlich unbedenklich sein, oder?

  • Aber was ist, wenn sie nur aus einzelnen Zeichen und CR/LFs usw. bestehen? Ich weiß es nicht. Ich denke, eine grobe Prüfung reicht aus; wir müssen uns hier keine Sorgen um extrem Randfälle machen.
1 „Gefällt mir“

Klar, wir können hier etwas machen, sogar nur mit CSS. Aber wir müssen die Bildlaufleiste reparieren, wenn wir diesen Ansatz wählen.

Andernfalls könnten wir es genauso gut in if 1 post then hide timeline ändern, was noch einfacher ist.

Es sei denn, dieser einzelne Beitrag ist eine freakische 30.000 Zeichen lange Sache, die wir nicht versteckt haben wollen, also… nein.

Wir möchten, dass es versteckt ist … tut mir leid … das ist mein Punkt. Es sei denn, wir beheben dieses Problem:

Ich ziehe die Zeitleiste nach unten:

Sie befindet sich immer noch oben. :crying_cat_face:

Diese Scrollleiste funktioniert jedoch wie erwartet, wenn ich die native Browser-Scrollleiste verwende:

Wenn wir sie für Themen mit einem riesigen Beitrag belassen, funktioniert sie einfach nicht richtig, was sehr verwirrend ist. Also müssen wir das beheben.

1 „Gefällt mir“

Ich verstehe, wir brauchen gute informelle Testfälle, um sicherzustellen, dass wir die abdecken, die uns wichtig sind.

1 „Gefällt mir“

Nein, das ist nicht der Commit.

Der betreffende Commit ist höchstwahrscheinlich:

Vor diesem Commit war die Logik gewissermaßen dupliziert und unterschiedlich und führte zudem zu einem leeren (aber sichtbaren) Timeline-Controls-Widget.

Ich werde mich heute darum kümmern.

3 „Gefällt mir“

Mein Versuch, das Überlappen der Zeitleiste zu beheben, ist hier:

Damit wird nicht das behoben, was du erklärt hast: Dass Klicken/Ziehen in der Zeitleiste die Scrollposition des Fensters nicht ändert. Ich habe das kurz angeschaut und denke, das sollte ein eigenes Projekt sein. Das war schon immer so, und eine Änderung hätte viele Auswirkungen.

Wenn wir nur einen Beitrag haben, überprüfe ich nun die Höhe des Beitrags und verstecke den Scrollbalken, wenn er weniger als 1000 px beträgt.

Das stellt einen Beitrag von ungefähr dieser Länge dar:

Beispiel mit einem kurzen Beitrag:

Dieser Commit behebt auch einen weiteren Fehler, der dazu führte, dass der Admin-Button für Themen nach einem vollständigen Seitenneuladen an einer falschen Position angezeigt wurde:

So:

Anstatt dieser Position nach der Korrektur:

Ich habe zudem überprüft, dass sich zwei Beiträge mit dem Mindestinhalt nicht überlappen:

Am Mobilgerät sollte sich nichts geändert haben.

4 „Gefällt mir“

Ausgezeichnet, danke, Joffrey :smiling_cat_with_heart_eyes:

2 „Gefällt mir“

Dies wurde zusammengeführt und kann beispielsweise in diesem Thema live betrachtet werden (zumindest solange es nur einen Beitrag gibt):

3 „Gefällt mir“

Dieses Thema wurde automatisch nach 5 Tagen geschlossen. Neue Antworten sind nicht mehr erlaubt.