Problème avec `topic-progress-wrapper` en RTL

Contribuer Bug
1

Bonjour cher Discourse,

En mode RTL, lorsque nous arrivons à la fin de la page du sujet, la classe topic-progress-wrapper rend la page encombrée. Veuillez examiner cet élément.

Lorsque nous commentons les valeurs suivantes, le problème est résolu :

@media screen and (min-width: 925px)
.container.posts .topic-navigation {
    grid-area: timeline;
    align-self: start;
    /* position: -webkit-sticky;
       position: sticky; */
    top: 6em;
    margin-right: 1em;
    z-index: 100;
}
2

Bonjour @Nildarar,

Pourriez-vous s’il vous plaît publier une capture d’écran du problème ?

3

Non visible dans la capture d’écran. Laissez-moi expliquer davantage. Pour voir le problème, veuillez suivre les étapes suivantes :

  1. Rendre votre modèle RTL
  2. Passer en vue mobile
  3. Augmenter la largeur de l’écran à plus de 925 px
  4. Faire défiler jusqu’à la fin du sujet
  5. Le module Progression du sujet disparaît de l’écran et la page défile horizontalement.

Nous aurions pu résoudre ce problème nous-mêmes en modifiant le CSS, mais j’ai pensé qu’il pourrait y avoir un bug au-delà de cela et qu’il serait détecté par votre examen.

1 « J'aime »
4

Ok, donc c’est un symptôme d’un problème plus large. Le problème racine ici est que Discourse devrait s’afficher en vue bureau plutôt qu’en vue mobile lorsque l’écran a une largeur ≥ 925 px. Pouvez-vous me dire sur quel(s) appareil(s) vous rencontrez ce problème ?

Il est important de mentionner que supprimer position: sticky peut sembler résoudre ce problème particulier, mais cela cassera la chronologie du sujet (elle ne défilera pas vers le bas lorsque vous faites défiler un sujet).

C’est un peu sans rapport avec cela, mais j’ai remarqué que la barre de progression du sujet devrait être à gauche plutôt qu’à droite en mode RTL. J’ai envoyé une PR à ce sujet ici : FIX: Topic progress bar should be on the left in RTL layouts by OsamaSayegh · Pull Request #11999 · discourse/discourse · GitHub.

7 « J'aime »
5

Des utilisateurs nous l’ont signalé et je l’ai constaté dans les outils de développement. Je vais demander à mes collègues et je te tiendrai informé.

Merci pour cette correction :+1:

2 « J'aime »
6

Fusionné, merci beaucoup pour le signalement. J’espère que cela résout le problème.

3 « J'aime »