Investigate scroll-offset management issues on pages with sticky banner
@pradyunsg がすでに取り組んでいます。
2023年3月20日 から。
- 主要言語
- reStructuredText
- スター
- 5k
- フォーク
- 1.8k
- 平均マージ
- 2日 4時間
- マージ済み PR(30日)
- 25
説明
In testing this, sometimes when loading a page with a fragment identifier (the original scenario motivating this change), at least for me in FF on Windows, the scroll offset isn't applied and the banner covers the heading, which I'm guessing is because the scroll event races this snippit on
DOMContentLoadedand sometimes the scroll is executed before the scroll offset has been set.
Originally posted by @CAM-Gerlach in https://github.com/python/peps/pull/2992#pullrequestreview-1276259998
Decided to defer the detailed investigation into this for a follow-up change -- this is an issue for tracking either the change of issuing an additional scroll event or using a different mechanism (eg: something render-blocking) to set the scroll offset instead or some other way to avoid the occasional race condition.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
評価
この issue はまだ評価されていません。