Problema con `topic-progress-wrapper` in RTL

Contribuisci Bug
1

Ciao Discourse,

In RTL, quando si arriva alla fine della pagina dell’argomento, la classe topic-progress-wrapper rende la pagina disordinata. Si prega di verificare questo elemento.

Quando si commentano i seguenti valori, il problema viene risolto:

@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

Ciao @Nildarar,

puoi pubblicare uno screenshot del problema?

3

Non mostrato nello screenshot. Lasciate che vi spieghi meglio. Per visualizzare il problema, seguite questi passaggi:

  1. impostate il vostro template su RTL
  2. entrate nella visualizzazione mobile
  3. aumentate la larghezza dello schermo a oltre 925px
  4. scorrete fino alla fine dell’argomento
  5. il modulo Progresso dell’argomento scompare dallo schermo e la pagina scorre orizzontalmente.

Avremmo potuto risolvere il problema noi stessi modificando il CSS, ma ho pensato che potesse esserci un bug più profondo e che sarebbe stato individuato dalla vostra revisione.

1 Mi Piace
4

Ok, quindi questo è un sintomo di un problema più ampio. Il problema di fondo qui è che Discourse dovrebbe essere visualizzato in modalità desktop anziché mobile quando lo schermo ha una larghezza ≥ 925px. Puoi dirmi su quale/i dispositivo/i vedi questo problema?

Vale la pena menzionare che rimuovere position: sticky potrebbe sembrare risolvere questo particolare problema, ma romperebbe la timeline dell’argomento (non scorrerebbe verso il basso mentre scorri l’argomento).

Quasi non correlato a questo, ma ho notato che la barra di progresso dell’argomento dovrebbe essere a sinistra invece che a destra quando si utilizza un layout RTL. Ho inviato una PR per questo qui: FIX: Topic progress bar should be on the left in RTL layouts by OsamaSayegh · Pull Request #11999 · discourse/discourse · GitHub.

7 Mi Piace
5

Gli utenti ce l’hanno segnalato e l’ho visto negli Strumenti per sviluppatori. Chiederò ai miei colleghi e ti farò sapere.

Grazie per questa correzione :+1:

2 Mi Piace
6

Fuso, grazie mille per la segnalazione. Spero che risolva il problema.

3 Mi Piace