[3.3] [CDB]Recent Topics NG

[IMC]

Ich hätte da zwar inzwischen eine Idee, mit der ich aber erstmal experimentieren muss,

Da bin ich mal gespannt.

Sehe ich auch so und ist unkompliziert umsetzbar und wäre dann auch konsequent.

Konsequenterweise müssten wir dies dann auch in "Wer ist online?" machen. Bei uns viewonline_overwrite_location().

Aktuelle Themen (NG)
---------------------------
  Einstellungen

Hier würde ich die Klammern weg lassen. Oder den gesamten eingeklammerten Ausdruck.

Aktuelle Themen
Auf dieser Seite kannst du die Einstellungen der Erweiterung „Recent Topics NG“ anpassen.

Ok.

Aber das ist Geschmackssache, die einen Ext Entwickler wollen den original Ext Namen, die anderen die Übersetzungen. Da gibts keine Vorgaben, das fällt alles unter künstlerische Freiheit und schreibt uns niemand vor.

Was meinst du dazu?

Ich gehöre eher zu den Leuten, die Eigennamen nicht übersetzen. Ich bin da aber auch flexibel. Mit dem Beispiel der französischen Übersetzung hast du mich überzeugt. Die sind etwas speziell, was Fremdsprachen betrifft.

Machst du alles, oder soll ich den Part "Unread Topics" übernehmen?

[LukeWCS]

Machst du alles, oder soll ich den Part "Unread Topics" übernehmen?

Ich fange direkt damit an. Mein erster Impuls gestern Abend nach deinem Beitrag war, dass du den Unread-Job übernimmst, weil du ja schon gesichtet hast, wo man ansetzen muss und ich dann den Titel-Job mache. Ich hätte nämlich WIO gleich mal übersehen gehabt, daran hatte ich nicht gedacht. Heute Morgen beim ersten Sichten der Sprachdateien und des Listeners ist mir aber klar geworden, dass das besser nur von einem gemacht wird, weil beide Jobs eine wichtige Schnittmenge haben und man sich bei separater Bearbeitung nur auf die Füsse treten würde, bzw. weil dann der eine etwas macht, was der andere wieder ändern muss. Darum würde ich das jetzt tatsächlich komplett machen wollen. Im Detail:

Hier würde ich die Klammern weg lassen.

Die habe ich bewusst gesetzt, weil das Kürzel ja von 2 englischen Wörtern abgeleitet ist und somit keinen direkten Bezug zum übersetzten RTNG Titel hat. Dasss das Kürzel bei deutscher Übersetzung ebenfalls gleichlauten würde, ist jetzt quasi nur Zufall und muss nicht für andere Sprachen gelten. Aber:

Oder den gesamten eingeklammerten Ausdruck.

Das! Wir lassen das Kürzel gänzlich weg, da kann ich gut mitgehen. Im Prinzip ist das Kürzel eh nicht relevant als Teil des Titels, weil es ein sehr spezifisches Merkmal in Bezug auf unseren Komplett-Umbau von RT darstellt und nichts mit der Funktion an sich zu tun hat.

Okay, also einheitlich "Aktuelle Themen" in Navi und Modul.

Mit dem Beispiel der französischen Übersetzung hast du mich überzeugt. Die sind etwas speziell, was Fremdsprachen betrifft.

Keine Widerrede meinerseits. Aber sind wir nicht alle ein wenig phpBB!?

Jetzt zur Schnittmenge:

Bedingt durch meinen Vorschlag mit der Lokalisierung des Navi- und Modul-Titels, ergeben sich weitere Änderungen in den Sprachdateien, die globalen Einfluss haben.

Hier die Änderungen, die ich gerne bei den Sprachdateien vornehmen würde:

• RTNG_NAME ist irreführend, weil das einerseits ein Titel, andererseits der Ext Name sein kann und beides muss nicht identisch sein. Das würde ich komplett entfernen und durch RTNG_TITLE ersetzen. Wenn wir wirklich mal den Namen brauchen, haben wir dafür die Template Variable RTNG_EXT_NAME, die ich ja erst eingebaut hatte. Durch diese Änderung hätten wir auch konsequent in allen Bereichen (ACP, UCP, Foren-Index, separate Seite, WIO) sowohl einen einheitlichen sichtbaren Titel, als auch eine einheitliche Sprachvariable dafür.
• Durch die obige Änderung wird RTNG_DESIG überflüssig und kann durchgehend ebenfalls durch RTNG_TITLE ersetzt werden. Wobei mir bei "DESIG" eh der Bezug fehlt, damit kann ich gerade nichts anfangen.
• Um auch das mit Unread unter einen Hut zu kriegen, würde ich zusätzlich die Variable RTNG_UNREAD_TITLE etablieren. Das kann z.B. direkt im Listener verdrahtet werden, dort habe ich bereits funktionierenden Code mittels einer schon existierenden Funktion unserer controller_common Klasse getestet, damit kann ich dann den Titel für WIO bei den Usern dynamisch anpassen. Oder hattest du was anderes im Sinn, wie das gelöst werden soll?

Ist das für dich plausibel?

[IMC]

dass du den Unread-Job übernimmst, weil du ja schon gesichtet hast, wo man ansetzen muss und ich dann den Titel-Job mache. Ich hätte nämlich WIO gleich mal übersehen gehabt, daran hatte ich nicht gedacht.

Ich auch nicht sofort. An das Breadcrumbs-Menue habe ich sogar erst heute Morgen gedacht. Dann werde ich das übernehmen.

Ist das für dich plausibel?

Ja, ist es.

[LukeWCS]

An das Breadcrumbs-Menue habe ich sogar erst heute Morgen gedacht.

Darauf bin ich dann heute Morgen beim Zurückverfolgen der Variablen auch gestossen.

Dann werde ich das übernehmen.

Jetzt hast du mich glaub ich missverstanden. Durch meine geplanten Änderungen fällt das mit Unread eh in meinen Bereich. Ich würde das also - wie von dir angefragt - komplett machen.

Oder willst du Unread selber machen? Dann müsstest du das zuerst machen, damit jeglicher Code (PHP) schon fertig ist, vorher kann ich nicht meine Änderungen bei den Sprachdateien machen.

[IMC]

Nee, dann mach du es. Dann ist alles in einer Hand.

[LukeWCS]

Sehe ich auch so und ist unkompliziert umsetzbar

Man sollte Aussagen zum Umfang einer Änderung/Neuerung erst dann machen, wenn man vorher auch den Source gesichtet hat.

Der erste Gedanke "Einfach eine neue Sprachvariable und mit einem Ternary auf die Einstellung verdrahten, fertig." hat sich schnell verflüchtigt, weil das an etlichen Stellen berücksichtigt werden muss und eben auch individuell angepasst werden musste, also keine Standardlösung für alle Seiten. Insbesondere bei WIO. Mittlerweile bin ich mit allem soweit fertig, ich muss nur noch deaktivierten Code und Debugs entfernen, dann kann ich einen Commit machen.

Hier mal ein Screen von WIO, da sieht man auch gleich die Anpassung für das Breadcrumb-Menü:
[ externes Bild ]

Ich bin davon abgerückt, RTNG_NAME zu entfernen, aus 2 Gründen:

1. Bei meinen Exts habe ich mir schon vor längerem angewöhnt, für die Navi eigene Sprachvariablen zu etablieren, damit ich Navi und Modul separat behandeln kann. Also maximale Flexibilität. Das nutze ich z.B. auch bei WWH, wo Navi Titel und Modul Titel unterschiedlich sind.
2. Es wäre zusätzlicher Aufwand per Migration notwendig, um diese Sprachvariable umzubenennen, weil man zuerst das Modul mit den alten Sprachvariablen entfernen und dann mit neuen Variablen wieder hinzufügen muss. Kann man machen, wenn es wirklich nötig ist, was ich hier aber nicht sehe.

Normal hätte ich jetzt die Variablen für die Navi nach meinem Schema in RTNG_NAV_TITLE und RTNG_NAV_CONFIG umbenannt, aber da nach meinem Umbau die Variablen RTNG_NAME und RTNG_CONFIG eh nur in der Navi genutzt werden, denke ich, können wir uns das mit der Migration sparen.

Dann, was mir aufgefallen ist, der Vorschlag von nou nou zielte auf "Beiträge" ab:

Small thing but it would be nice if when "show only Unread Posts" is on, the title on the Index would "Unread Posts", instead of "Recent Topics"?

Mein erster Impuls war, dem zuzustimmen, aber das würde dann mit der Standard Funktion von phpBB kollidieren, die genauso heisst. Ich vermute deswegen hast du dich für "Ungelesene Themen" entschieden? Das wäre technisch zwar nicht ganz korrekt, aber würde erstens noch am ehesten zu "Aktuelle Themen" passen und zweitens keine Verwechslungsgefahr mit sich bringen. Lassen wir so?

[IMC]

Der erste Gedanke "Einfach eine neue Sprachvariable und mit einem Ternary auf die Einstellung verdrahten, fertig." hat sich schnell verflüchtigt,

Einfach kann doch jeder. Wir hatte doch schon gut vorgearbeitet. Mit unseren Funktionen können wir unkompliziert für jede User-Id die Settings ermitteln. Was für WiO wichtig ist.

Mein erster Impuls war, dem zuzustimmen, aber das würde dann mit der Standard Funktion von phpBB kollidieren, die genauso heisst. Ich vermute deswegen hast du dich für "Ungelesene Themen" entschieden? .... Lassen wir so?

Wenn man es ganz genau nimmt, zeigen wir mit der Einstellung Nur ungelesene Themen anzeigen Themen an, die ungelesene Beiträge enthalten. (Identisch zu phpBB. Nur dort heist es ungelesene Beiträge.)

Mit der Einstellung Link des Thementitels zu ungelesenen Beiträgen würden wir ungelesene Beiträge anzeigen.

Um die ganze Sache nicht unnötig zu verkomplizieren hatte ich mich für "Ungelesene Themen" entschieden. Auch vor dem Hintergrund einen klaren Bezug zu unsere Erweiterung zu haben.

Aus meiner Sicht sollten wir es so lassen. Die Bezeichnung ist weder bei phpBB, noch bei uns 100%tig richtig. Durch die unterschiedliche Bezeichnung ist aber klar zu erkennen um welche Anzeige es geht.

[LukeWCS]

Wir hatte doch schon gut vorgearbeitet. Mit unseren Funktionen können wir unkompliziert für jede User-Id die Settings ermitteln. Was für WiO wichtig ist.

Und mit genau der habe ich auch gearbeitet, damit war das prima lösbar.

Um die ganze Sache nicht unnötig zu verkomplizieren hatte ich mich für "Ungelesene Themen" entschieden. Auch vor dem Hintergrund einen klaren Bezug zu unsere Erweiterung zu haben.

Eben das war meine Einschätzung.

Aus meiner Sicht sollten wir es so lassen.

Jupp, sehe ich auch so und ich habe auch schon mal vorab bereits alles darauf ausgelegt. Okay, dann kann ich mit Bereinigung anfangen.

[IMC]

Hi Patrick,

weil du gerade beim Template bei bist.


Thema: https://www.phpbb.com/customise/db/exte ... 63#p905263

When I add the add-on to the top or bottom, it looks fine and normal. When I add it to the sidebar, the links are smaller.

Dabei geht es um die Anzeigegröße des Thementitels a.topictitle. Ich habe mal nachgeschaut warum dieser in der seitlichen Anzeige kleiner angezeigt wird. In der "normalen" Ansicht erbt der Titel von ul.topiclist dt mit einer Grösse von 1.1em. Das bedeute das der Titel eine Anzeigegröße von 1.1em x 1.2em = 1.32em hat. Da der Titel in der seitlichen Ansicht sich nur auf den <body> bezieht wird er nur mit 1.2em angezeigt.
Kannst du das bitte gleich mit ändern / hinzufügen.

Code: Alles auswählen

#rtng-side a.topictitle {
	font-size: 1.3em;
} 

Thema: https://www.phpbb.com/customise/db/exte ... 78#p905278

I see that when the topic list is on the right, in mobile view it loads at the top of the forum, not at the bottom as it would theoretically appear.

It seems that what's at the top is at the top, what's on the left is at the top, but what's on the right should be at the bottom – you know what I mean.

I will discuss this with my co-author. Then we will decide whether to change it. The decision will take some time.

Das Thema hat bei mir im Moment nicht die höchste Priorität. Trotz allem hat er Recht. Ich bin mir sicher das viele RTNG lieber Oben haben möchten um einen guten Überblick über die Themen zu haben. Wenn wir etwas ändern wollen, wird es keine einfache Lösung geben.

[LukeWCS]

Wegen Titelgrösse der Seitenansicht; machst du bitte? Ich hab gestern Nacht dev16 schon hochgeladen und wäre damit fertig, wenn ich keinen Fehler gemacht habe.

Was Responsive und Seitenansicht angeht, gebe ich ihm Recht, logisch ist das aktuelle Verhalten wirklich nicht.

Hier gibts aber noch anderes zu berücksichtigen: RT hat eine lange Tradition und die Seitenansicht gabs ja nicht erst bei RTNG. Obs das auch schon beim Mod gab weiss ich nicht, ist aber auch nicht mehr relevant. Die Admins die die Seitenansicht nutzen, sind das Verhalten seit eh und je so gewöhnt und ich bin ebenfalls ziemlich sicher, das die meisten RT Nutzer die Anzeige generell oben haben wollen, egal ob PC oder Smartphone.

Aber wie auch immer. Wir ändern das nicht generell, nur weil einem das jetzt nicht gefällt, da sonst vermutlich viele andere meckern würden. Wenn wir da was machen sollten, dann müsste das a) per Konfig frei wählbar sein und b) standardmässig das gewohnte Verhalten haben.

Dann, die Seitenansicht ist eh "hakelig", weil da die problematische Float-Technik genutzt wird, die ich selber meide wo immer ich kann. Ich nutze diese Technik nur sehr selten und nur da, wo sie nicht gleich ein heilloses Chaos verursachen kann, wenn mal was nicht 100% passt. Was schnell passieren kann, wenn einem nicht-isoliertes CSS von einer anderen Ext reingrätscht.