Display the timer on the header next to the topic title.
I’m looking for specific UI/UX suggestions for how to implement a header timer, in the context of Discourse, with a view to crowd-funding the development costs if there’s enough interest (@darhazer@tobiaseigen)
Derzeit funktioniert es für alle zeitgesteuerten Themen. @ninjapenguin, wenn du bereit bist, die Entwicklung dieses Features zu finanzieren, füge ich gerne die kategoriespezifische Funktion für die Komponente hinzu.
In jedem Fall kannst du dich hier an der Diskussion zur Komponente beteiligen:
Dieses Theme-Komponente stört leider die Benutzeroberfläche für Themen-Timer. Die Anzahl der Tage scheint in die Anzahl der Sekunden umgewandelt zu werden, und die blaue Leiste mit dem Text „Antworten in diesem Thema werden automatisch gelöscht nach ..“ hat keinen Fortsetzungsabschnitt. Dadurch ist es unmöglich, den Timer zu entfernen. Das Löschen dieser Theme-Komponente hat die Funktion wiederhergestellt.
Es muss erneut abgestaubt werden. Ich werde mich nächste Woche darum kümmern. Danke für den Hinweis. Das unterstreicht für mich die Notwendigkeit von Lesezeichen-Erinnerungen.
Ich kann das Problem reproduzieren. Entschuldigung, dass ich nicht früher darauf eingegangen bin.
Das Theme-Modul wurde nie mit anderen Timern als dem close topic timer getestet, da es speziell für diesen Timer entwickelt wurde. Der Name legt jedoch nahe, dass es für alle Arten von Themen-Timern funktionieren sollte.
Außerdem sollte es zumindest keine anderen Standardfunktionen beeinträchtigt haben. Ich arbeite daran, das Problem zu beheben, und werde bald mit einer Lösung zurückkommen.
Dies war meine erste Client-Arbeit bei Discourse. Es weckt alte Erinnerungen.
p.s.
Ich habe einen PR an den Core-Topic-Timer eingereicht, um ihn für Plugin- und Theme-Entwickler erweiterbar zu machen. Das Überschreiben der Monolith-Methode kann sehr fehleranfällig sein.
Das war ein schlechtes Erinnerungsstück meinerseits. Ich habe fälschlicherweise einen Beitrag von Jeff im Kopf, der diese Begriffe erklärt, aber ich kann mir vorstellen, dass IMPR nicht dabei war. Ich werde mir das merken.
Die gute Nachricht ist, dass der Code nach dem Merge aus Entwicklersicht jetzt erweiterbarer und wartbarer ist. Ich werde versuchen, die Komponente in der kommenden Woche zu reparieren.
Danke @sam für den Hinweis und @martin für den Merge.
