Problema com `topic-progress-wrapper` em RTL

Contribuir Bug
1

Olá, querido Discourse,

Em layouts RTL, ao chegar ao final da página do tópico, a classe topic-progress-wrapper deixa a página desorganizada. Por favor, verifique este item.

Ao comentar os seguintes valores, o problema é resolvido:

@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

Olá @Nildarar

Você pode, por favor, postar uma captura de tela do problema?

3

Não mostrado na captura de tela. Deixe-me explicar melhor. Para ver o problema, faça o seguinte:

  1. Configure seu modelo para RTL
  2. Entre na visualização móvel
  3. Aumente a largura da tela para mais de 925px
  4. Role até o final do tópico
  5. O módulo Progresso do Tópico desaparece da tela e a página rola horizontalmente.

Poderíamos ter corrigido esse problema nós mesmos alterando o CSS, mas achei que poderia haver um erro além disso e que seria identificado na sua revisão.

1 curtida
4

Ok, então esse é um sintoma de um problema maior. O problema raiz aqui é que o Discourse deve renderizar na visualização de desktop, e não na visualização móvel, quando a tela tiver ≥ 925px de largura. Você pode me dizer em qual(is) dispositivo(s) você vê esse problema?

Vale mencionar que remover position: sticky pode parecer resolver esse problema específico, mas vai quebrar a linha do tempo do tópico (ela não descerá conforme você rola o tópico para baixo).

Um pouco fora do contexto disso, mas notei que a barra de progresso do tópico deveria estar à esquerda, e não à direita, em layouts RTL. Enviei um PR para isso aqui: FIX: Topic progress bar should be on the left in RTL layouts by OsamaSayegh · Pull Request #11999 · discourse/discourse · GitHub.

7 curtidas
5

Usuários relataram isso para nós e eu vi isso nas Ferramentas de Desenvolvedor. Vou perguntar aos meus colegas e te aviso.

Obrigado por essa correção :+1:

2 curtidas
6

Mesclado, muito obrigado pelo relato. Espero que isso resolva o problema.

3 curtidas