Argomenti saltano al post sbagliato

Grazie mille per i chiari passaggi di riproduzione @Don :pray:

Sì, è proprio così! Ci sono due cose che accadono contemporaneamente

  1. Il servizio di caricamento dello slider deve rimuovere la classe still-loading dal body
  2. LockOn deve scorrere nel punto giusto

Entrambi erano programmati nella parte afterRender del runloop di Ember. E poiché le cose di ‘lock on’ sono tecnicamente programmate per prime, sono state eseguite per prime. E così LockOn è stato eseguito mentre tutto l’HTML del post era nel DOM, ma aveva ancora display: none. :grimacing:

Questo PR sposterà la rimozione della classe still-loading nella parte ‘render’ del runloop, il che significa che qualsiasi altra cosa che programma qualcosa in afterRender verrà eseguita una volta che tutto sarà renderizzato e visibile:

D’accordo! Non volevo toccarlo come parte di questa correzione di bug, ma penso che dovremmo puntare a rimuovere tutte quelle soluzioni di fortuna.

Leggendo i commenti nel file, sembra che sia stato originariamente introdotto (10 anni fa!) per contrastare le funzionalità di ‘ripristino dello scorrimento’ del browser. Al giorno d’oggi, usiamo history.scrollRestoration = false per disabilitare quella funzionalità del browser, quindi penso che ciò renda ridondanti molte delle vecchie soluzioni di fortuna.

Probabilmente è meglio provare questo tipo di modifica sensibile tramite un componente tema per prima cosa, e poi se tutto sembra a posto potremo unirlo al core. Immagino che la rimozione di LockOn risolverà molti altri casi limite che abbiamo con la posizione di scorrimento dei topic. Cercherò di provarlo nelle prossime settimane :technologist:

7 Mi Piace