Hi.
On short topics, the timeline is overlapping the “Suggested Topics” section:
I guess the timeline could be hidden on topics without any replies, or the posts container minimum height should be about the same as the timeline?
4 Mi Piace
As I recall, this has been around a while and is endemic to short screens and single-post one line topics, but perhaps we regressed? 
Assolutamente, lo sto notando anch’io.
@awesomerobot mi ha ricordato che in passato disattivavamo la timeline per gli argomenti con un solo post, credo? Forse è per questo che la stiamo notando di più, oppure si tratta di un vero e proprio regresso? In ogni caso, in questo scenario, tutti gli elementi della timeline sono duplicati nei controlli in fondo al primo (e unico) post…
1 Mi Piace
Sì, penso che lo stiamo nascondendo dal 2016: UX: Hide all timeline controls when fewer than 3 posts · discourse/discourse@b9f8264 · GitHub
Fino a non molto tempo fa, la cronologia nascondeva
@j.jaffeux, potrebbe essere la tua modifica recente a aver causato questa regressione?
3 Mi Piace
Dovremmo sicuramente nascondere la timeline quando c’è un solo post. Tre sembrano un po’ troppi. Ma uno…
1 Mi Piace
Il grande problema con un singolo post è che, quando è effettivamente utile come questo, non funziona correttamente. Non supportiamo la navigazione a metà di un post enorme, quindi diventa estremamente confuso. Molto peggio del semplice problema di rendering.
2 Mi Piace
Fallo in base alla lunghezza del post più il singolo post allora… non abbiamo bisogno di una timeline verticale per un post singolo di una frase in un singolo argomento, vero?
Assolutamente no, ma dobbiamo sistemare l’implementazione, immagino sia questo a cui sto cercando di arrivare.
Se dovessi scegliere tra rimuoverlo incondizionatamente o mantenerlo per post singoli eccezionalmente lunghi, sceglierei di rimuoverlo incondizionatamente.
Se però risolviamo l’implementazione sottostante, sarei totalmente a favore di mantenerlo per la guida straordinaria di @Johani; sarebbe davvero utile.
La logica dovrebbe essere:
se c’è un solo post nel topic E se il singolo post ha meno di .. 1000* caratteri .. sopprimi completamente la timeline
Anche due post di una sola frase sono probabilmente sicuri, forse?
- ma cosa succede se sono tutti caratteri singoli e CR/LF ecc. non lo so. Penso che un controllo approssimativo vada bene, non dobbiamo preoccuparci di casi limite assurdi qui.
1 Mi Piace
Certo, possiamo fare qualcosa qui usando anche solo CSS. Ma dobbiamo correggere la barra di scorrimento se scegliamo questo approccio.
Altrimenti, tanto vale cambiarlo in if 1 post then hide timeline, che è ancora più semplice.
Tranne se quel singolo post è una mostruosità di 30.000 caratteri, non vogliamo che sia nascosto, quindi… no.
Vogliamo proprio che sia nascosto… scusa… questo è il mio punto. A meno che non risolviamo questo problema:
Trascino la timeline verso il basso:
Rimane comunque in alto. 
Questa barra di scorrimento invece funziona come previsto se uso quella nativa del browser:
Se la lasciamo così per i topic con un singolo post enorme, semplicemente non funziona correttamente, il che è molto confuso. Quindi dobbiamo risolvere quel problema.
1 Mi Piace
Capisco, servono buoni casi di test informali per assicurarci di aver coperto quelli che ci interessano.
1 Mi Piace
No, questo non è il commit.
Il commit correlato è molto probabilmente:
Quindi, prima di questo commit, la logica era in qualche modo duplicata e diversa, e causava anche un widget timeline-controls vuoto (ma visibile).
Ci darò un’occhiata oggi.
3 Mi Piace
Il mio tentativo di risolvere l’overlap della timeline è qui:
Non risolve quanto hai spiegato, ovvero che cliccare/trascinare sulla timeline non modifica la posizione di scorrimento della finestra. Ho dato una rapida occhiata a come implementarlo e penso che questo dovrebbe essere un progetto dedicato: è sempre stato così e cambiarlo avrebbe MOLTE implicazioni.
Se c’è un solo post, ora controllerò l’altezza del post e nasconderò lo scrollbar se è inferiore a 1000px.
Rappresenta un post di circa questa lunghezza:
Esempio con un post breve:
Questo commit risolve anche un altro bug, che causava la visualizzazione del pulsante di amministrazione del topic in una posizione errata dopo un aggiornamento completo della pagina:
Come questo:
Invece di questa posizione dopo la correzione:
Ho anche verificato che due post con la quantità minima di contenuto non si sovrappongano:
Nulla dovrebbe essere cambiato su mobile.
4 Mi Piace
Ottimo, grazie Joffrey 
2 Mi Piace
Questo è stato unito e può essere visto in azione in questo argomento, per esempio (almeno finché c’è solo un post):
3 Mi Piace
Questo argomento è stato automaticamente chiuso dopo 5 giorni. Non sono più consentite nuove risposte.