Zeitachne überlappt „Vorgeschlagene Themen"

Hi.
On short topics, the timeline is overlapping the “Suggested Topics” section:


I guess the timeline could be hidden on topics without any replies, or the posts container minimum height should be about the same as the timeline?

4 „Gefällt mir“

As I recall, this has been around a while and is endemic to short screens and single-post one line topics, but perhaps we regressed? :thinking:

It also happen when long enough content and has some content above the post

Definitely seeing this.

@awesomerobot reminded me that we used to suppress the timeline for single post topics, I think? Perhaps that’s why we’re seeing it more, perhaps it is an actual regression? All the timeline elements are duplicated in the controls at the bottom of the first (and only) post in this case, anyway…

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.