トピックが間違った投稿にジャンプする

バグ
31

@Don、明確な再現手順をありがとうございます :pray:

はい、その通りです！同時に2つのことが起こっていました。

  1. Loading slider service は still-loading クラスを body から削除する必要があります。
  2. LockOn は正しい位置にスクロールする必要があります。

どちらも Ember の runloop の afterRender 部分でスケジュールされていました。そして、「lock on」関連の処理は技術的には最初にスケジュールされていたため、最初に実行されていました。そのため、LockOn は投稿の HTML がすべて DOM に存在している間に実行されていましたが、まだ display: none の状態でした。:grimacing:

この PR は、still-loading クラスの削除を runloop の「render」部分に移動します。これにより、「afterRender」で何かをスケジュールしている他のものはすべて、すべてがレンダリングされ表示された後に実行されるようになります。

同意します！このバグ修正の一部として、それに触れたくはありませんでしたが、これらのハックをすべて削除することを目指すべきだと思います。

ファイル内のコメントを読むと、元々は（10年前！）ブラウザの「スクロール復元」機能に対抗するために導入されたようです。現在では、 history.scrollRestoration = false を使用して そのブラウザ機能を無効にしているため、古いハックのほとんどは冗長になっていると思います。

おそらく、このような機密性の高い変更は、まずテーマコンポーネントを通じて試行するのが最善でしょう。そして、すべてがうまくいけば、コアにマージできます。LockOn を削除することで、トピックのスクロール位置に関する他の多くのエッジケースが修正されると想像しています。来週か再来週には試してみるつもりです :technologist:

「いいね！」 7