Come riprodurre:
Credo che il problema sia causato da questo codice.
Riuscite a riprodurre questo problema qui su meta o su try.discourse.org?
Io riesco a riprodurlo su Feverbee, ma non qui. Potrebbe essere dovuto alle personalizzazioni che hanno implementato, se non riusciamo a riprodurlo qui.
Puoi riprodurlo qui:
(riduci la larghezza, ma non troppo)
Questo è davvero un bug… se ci sono solo 2 post nell’argomento e si riduce la larghezza dello schermo, il problema si verifica.
Concordo sul fatto che il codice collegato sia probabilmente la causa.
Il link dovrebbe rimanere stabile, dato che fa parte dei dati seed.
Potresti dare un’occhiata a questa @j.jaffeux?
Quindi sì, questo codice è responsabile e si verifica in base a un rapporto specifico tra l’altezza della finestra e quella del topic.
Ma ciò che non riesco a capire è: qual è il comportamento atteso? Potrei semplicemente rimuovere questo controllo e funzionerebbe, ma immagino che l’abbiamo aggiunto per un motivo?
@eviltrout hai implementato questa funzionalità iniziale FIX: If a topic is really long, show the timeline · discourse/discourse@79763a6 · GitHub
Non è più valida? La formula è errata? Penso che si stia attivando qui anche quando il topic non è così grande, quindi forse dovremmo correggerlo?
Inoltre, ho scoperto che tutti i topic suggeriti venivano utilizzati nel calcolo dell’altezza del topic: ${"#topic").css("background", "red"):
Forse è questo che è cambiato da quando l’hai implementata.
Forse dovremmo anche risolverlo prima? Perché mostrare il progresso dell’argomento in questo caso?
Se è <= 3 post, nascondiamo il progresso dell’argomento se posts.height < window.height * 1.5?
In origine, la logica era quella di non mostrare il widget di avanzamento se l’argomento è davvero breve, cosa che ritengo abbia ancora senso.
Tuttavia, avevo l’impressione che il bug fosse il fatto che il widget venga visualizzato, ma cliccandoci sopra non funzioni. Una soluzione adeguata sarebbe far sì che il clic funzioni ogni volta che il widget viene mostrato. Sono d’accordo nel mantenerlo nascosto in altri momenti (per argomenti brevi).
Mi chiedo se dovremmo semplicemente impostare la soglia di nascondimento a 1, almeno su mobile / in modalità collassata (o addirittura rimuovere questa logica da mobile / in modalità collassata).
Su mobile, anche se hai solo 2 post, aiuta molto a capire se ti trovi nel primo o nel secondo.
C’è un valore nel mostrare sempre la timeline anche su desktop, ad esempio se il post che stai leggendo non entra in una schermata.
Dovrebbe risolvere il problema:
Dettagli nel commit.
Questo argomento è stato automaticamente chiuso 5 giorni dopo l’ultima risposta. Non sono più consentite nuove risposte.
