hosna
(hosna)
Mai 11, 2023, 4:19
1
Auparavant, les avatars étaient à droite. C’était mieux.
2 « J'aime »
Lilly
(Lillian Louis)
Mai 11, 2023, 4:25
2
Peut-être essayer le mode sans échec ? Cela ressemble à un problème avec un composant ou quelque chose qui n’est pas d’accord avec une mise à jour ?
hosna
(hosna)
Mai 11, 2023, 4:41
3
Non
C’est toujours comme ça.
Je peux reproduire cela sur mon site de test également lorsque je le change en locale RTL. Quelque chose semble effectivement être de travers. Laissez-moi voir si je peux demander à quelqu’un d’examiner cela de plus près.
5 « J'aime »
hosna
(hosna)
Mai 18, 2023, 6:22
8
Des nouvelles à ce sujet ?
La bonne nouvelle, c’est que nous avons identifié l’origine du problème et qu’un ingénieur a été assigné pour s’en occuper.
Je ne suis pas sûr du délai pour la correction, mais j’espère que ce ne sera pas trop long.
3 « J'aime »
Osama
Mai 29, 2023, 10:36
10
Voici une PR pour corriger ce bug :
main ← fix/layout-problems-rtl
opened 10:27PM - 29 May 23 UTC
We currently have some CSS rules in `common/base/rtl.scss` that were added to wo… rkaround shortcomings of the R2 gem that we used to use to generate a version of our CSS that's suitable for RTL layouts. Those workarounds are mostly duplicate rules with the only difference being that they're flipped to suit RTL layouts.
However, we've recently replaced R2 with `rtlcss` which doesn't have those shortcomings of R2 (see https://github.com/discourse/discourse/commit/f94951147ea9d1c48ea68f0dabd568e70cb94674) which means those workarounds/duplicate rules need to be removed because they're getting flipped by `rltcss`, essentially reverting them to their original LTR version and causing issues with RTL layouts.
This PR removes those workarounds that are no longer needed, and cleans up the the file that contains our RTL-specific CSS.
Meta topic: https://meta.discourse.org/t/avatar-in-rtl-website-in-wrong-place/264676?u=osama.
(elle n’est pas encore fusionnée, mais devrait l’être cette semaine)
Merci d’avoir signalé ce bug @hosna !
5 « J'aime »