Layout der Seite `Admin  Badges` nach Update defekt

Hallo zusammen.

Ich habe gerade auf Disourse 3.4.0.beta2-dev (cde4db1c6a) aktualisiert und festgestellt, dass das Layout der Admin-Seite für Badges etwas kaputt ist.

Die Liste der Badges befindet sich normalerweise auf der linken Seite des Bildschirms, sodass Sie die Badges auf der rechten Seite verwalten / hinzufügen / bearbeiten können.

Aber so sieht es jetzt aus, es gibt einen großen leeren und unbrauchbaren Bereich in der linken Mitte:

1 „Gefällt mir“

Können Sie den abgesicherten Modus ausprobieren? Ich kann das nicht reproduzieren. :thinking:

1 „Gefällt mir“

Hmm, es tritt sowohl in Safari als auch in Chrome auf, beides getestet und auch im abgesicherten Modus reproduziert.

Ok, danke @Lilly - es könnte etwas anderes sein, das ich woanders geändert habe, du kannst das hier wahrscheinlich ignorieren :blush:

1 „Gefällt mir“

Ich habe gerade eine frische Bereitstellung auf meiner gehosteten Testseite vorgenommen und die Badge-Verwaltung funktioniert wie erwartet. Es ist seltsam, dass dies im abgesicherten Modus geschieht.

Ich habe auch eine Überprüfung mit dem Dropdown-Menü in der Kopfzeile anstelle der Seitenleiste durchgeführt, sowie mit dem alten Admin-Navigationsmenü, und es funktioniert immer noch wie erwartet.

4 „Gefällt mir“

Schritte zur Reproduktion:

  1. Ändern Sie die Einstellung Standard-Locale auf Englisch (UK):

  1. Laden Sie Discourse in einem neuen Tab neu und navigieren Sie zu https://example.com/admin/badges, wo das Layout nun fehlerhaft ist:

  1. Ändern oder setzen Sie die Einstellung Standard-Locale auf den Standardwert Englisch (US) zurück:

  1. Laden Sie Discourse in einem neuen Tab neu und navigieren Sie zu https://example.com/admin/badges, wo das Layout nun einwandfrei funktioniert:

Ich habe absolut keine Ahnung, wie oder warum das so ist :thinking:

Aber ein großes Dankeschön an @JammyDodger für die Hervorhebung der Ursache :blush:

Könnte dieser Fehler behoben werden? :thinking:

4 „Gefällt mir“

Danke @Richie!

Es scheint, als gäbe es hier ein paar Probleme.

  1. Der englischen (UK) Übersetzung fehlt die neue, längere Beschreibung am Anfang der Seite. Das liegt daran, dass der Schlüssel ‘badges.description’ wiederverwendet wurde und wir seitdem kein Update von Crowdin mehr durchgeführt haben.

    Dies wird sich in Kürze von selbst lösen, wenn die Übersetzungen automatisch aktualisiert werden.

  2. Das Layout scheint von der Länge der Beschreibung abzuhängen. Im US-Englischen kann ich das Problem reproduzieren, indem ich die Entwicklertools öffne und die Beschreibung verkürze.

    Selbst wenn der String für das Vereinigte Königreich-Englisch aktualisiert ist, ist dies immer noch eine Reparatur wert, da andere Sprachen, die weniger Zeichen verwenden, wahrscheinlich das gleiche Problem haben werden.

Es sieht so aus, als ob die Änderung mit UX: Apply admin UI to Badges (#28724) · discourse/discourse@be5c37a · GitHub zusammenhängt (cc @ella @martin)

6 „Gefällt mir“

Danke für die Einblicke @david :smiley:

4 „Gefällt mir“

Ich bin in Kanada und gebe dem Vereinigten Königreich die Schuld dafür. :laughing:

Danke Richie. Ich habe die Locale-Einstellung gar nicht berücksichtigt :woman_facepalming:t2:

5 „Gefällt mir“

Das ist völlig themenfremd, aber es gibt kein Englisch (CA), oder? Würden Sie also britische oder US-amerikanische Lokalisierungen verwenden? Es mag natürlich davon abhängen, aber im Allgemeinen?

3 „Gefällt mir“

Unsere Sachen sind normalerweise auf die USA voreingestellt. Wir sagen hier nicht viel „Loo“ oder „Trousers“. Lol

4 „Gefällt mir“

Kein Problem, wir übernehmen diese gerne im Namen des Rests der Welt :raised_hand:t2: :blush:

4 „Gefällt mir“

Danke, dass Sie das bemerkt haben!

Ich habe ein Div-Wrapper hinzugefügt, um das Layout stabiler zu machen, und das Problem mit Inkonsistenzen bei der Verwendung verschiedener Lokalisierungen behoben.

Sollte mit diesem PR behoben sein:

6 „Gefällt mir“

Mein ganzes Land dankt dir für die schnelle Lösung @ella :bowing_man:

3 „Gefällt mir“