Disposition de la page `Admin bb Badges` casse9e apre8s la mise e0 jour

Salut tout le monde.

Je viens de mettre à jour vers Disourse 3.4.0.beta2-dev (cde4db1c6a) pour constater que la mise en page de la page d’administration des Badges est un peu cassée.

La liste des badges se trouve habituellement sur le côté gauche de l’écran, vous permettant de gérer / ajouter / modifier les badges sur le côté droit de l’écran.

Mais voici à quoi cela ressemble maintenant, il y a un grand espace vide et inutilisable au centre gauche :

1 « J'aime »

Pouvez-vous essayer le mode sans échec ? Je ne parviens pas à reproduire cela. :thinking:

1 « J'aime »

Hmm, ça se produit sur Safari et Chrome, les deux ont été testés et reproduits en mode sans échec aussi.

Ok, merci @Lilly - c’est peut-être autre chose que j’ai changé ailleurs, tu peux probablement ignorer celui-ci :blush:

1 « J'aime »

Je viens de faire un nouveau déploiement sur mon site d’hébergement de test et l’administration des badges fonctionne comme prévu. Il est étrange que cela se produise en mode sans échec.

J’ai également vérifié avec le menu déroulant d’en-tête au lieu de la barre latérale, ainsi qu’en utilisant l’ancien menu de navigation d’administration, et cela fonctionne toujours comme prévu.

4 « J'aime »

Étapes pour reproduire :

  1. Changez le paramètre locale par défaut en Anglais (Royaume-Uni) :

  1. Rechargez Discourse dans un nouvel onglet et accédez à https://example.com/admin/badges où la mise en page est maintenant cassée :

  1. Changez ou réinitialisez le paramètre locale par défaut à son état par défaut Anglais (États-Unis) :

  1. Rechargez Discourse dans un nouvel onglet et accédez à https://example.com/admin/badges où la mise en page fonctionne maintenant parfaitement :

Je n’ai absolument aucune idée de comment ou pourquoi c’est le cas :thinking:

Mais un grand bravo à @JammyDodger pour avoir mis en évidence la cause profonde :blush:

Ce bug pourrait-il être corrigé ? :thinking:

4 « J'aime »

Merci @Richie !

Il semble y avoir quelques problèmes ici.

  1. La traduction en anglais (Royaume-Uni) manque de la nouvelle description plus longue en haut de la page. C’est parce que la clé ‘badges.description’ a été réutilisée, et nous n’avons pas effectué de mise à jour depuis Crowdin depuis lors.

    Cela se résoudra imminemment, lorsque les traductions seront mises à jour automatiquement.

  2. La mise en page semble dépendre de la longueur de la description. En anglais américain, je peux reproduire le problème en allant dans les outils de développement et en raccourcissant la description.

    Même une fois la chaîne en anglais britannique mise à jour, cela vaut toujours la peine d’être corrigé car d’autres langues qui utilisent moins de caractères risquent d’avoir le même problème.

Il semble que le changement soit lié à UX: Apply admin UI to Badges (#28724) · discourse/discourse@be5c37a · GitHub (cc @ella @martin)

6 « J'aime »

Merci pour cet aperçu @david :smiley:

4 « J'aime »

Je suis au Canada et je blâme le Royaume-Uni pour ça. :rire :

merci Richie. Je n’avais même pas pensé au paramètre régional :visage_se_frappant_le_front:t2:

5 « J'aime »

Ceci est totalement hors sujet, mais il n’y a pas d’anglais (CA), n’est-ce pas ? Alors utiliseriez-vous les paramètres régionaux du Royaume-Uni ou des États-Unis ? Cela peut dépendre bien sûr, mais en général ?

3 « J'aime »

nos affaires sont généralement par défaut aux États-Unis. nous ne disons pas beaucoup « loo » ou « trousers » ici. lol

4 « J'aime »

Ce n’est pas un problème, nous revendiquerons volontiers la propriété de celui-ci au nom du reste du monde :raised_hand:t2: :blush:

4 « J'aime »

Merci d’avoir repéré ça !

J’ai ajouté un wrapper div pour rendre la mise en page plus stable et j’ai corrigé le problème d’incohérence lors de l’utilisation de différentes locales.

Devrait être corrigé avec cette PR :

6 « J'aime »

Mon pays tout entier te remercie pour la correction rapide @ella :bowing_man:

3 « J'aime »