WebKit: removing an animating table row throws in deletePosition()
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 36/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript, typescript, web-dev
調査の方向性
The report identifies animate() and deletePosition() as the relevant entry points and links minimal regression CI runs where Chromium passes and WebKit fails. Start by tracing how ancestor coordinates are cached and used when the exit animation is created. Done means the regression passes in both Chromium and WebKit without leaving a ghost row or blocking later mutations.
索引モデルが issue の本文から書いたものです。
説明
Removing a <tr> during its entry animation throws in deletePosition() in WebKit. Reproduced with 0.10.0, native DOM operations and default options; no framework or positioning changes are needed.
Reproduction
<div style="position: relative">
<table><tbody id="rows"><tr><td>Keep this row</td></tr></tbody></table>
</div>
import autoAnimate from '@formkit/auto-animate'
const rows = document.querySelector('#rows')
autoAnimate(rows)
const row = rows.insertRow()
row.insertCell().textContent = 'Remove this row'
while (!row.getAnimations().some(a => a.playState === 'running')) {
await new Promise(requestAnimationFrame)
}
row.remove()
Expected: the removed row is cleaned up and its sibling survives.
Actual in WebKit: undefined is not an object (evaluating 'parentCoords.top'). The row is reinserted with __aa_del before the exception, leaving a ghost row and preventing subsequent mutation handling.
Root cause
WebKit explicitly resets relative positioning to static on table row groups, including tbody. AutoAnimate therefore finds the relative outer wrapper, which it does not observe and has not cached.
animate() cancels the entry animation but keeps its registry entry in the idle state. deletePosition() then selects coords.get(offsetParent)! and reads .top from undefined. Measuring the ancestor when its cache entry is missing resolves this invalid cache assumption.
Related reports checked: #7 (table layout), #231 (ghost rows), #242 and #247 (cleanup). This report isolates the missing ancestor-coordinate entry before the exit animation is created. Fix in #249.
Minimal regression CI, unpatched core: Chromium passes, WebKit fails with the exception above. Same test with the patch: both pass.
- 主要言語
- TypeScript
- スター
- 13.9k
- フォーク
- 266
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
formkit/auto-animate のほかの issue
-
0.10.0: destroy() can leave an IntersectionObserver on a detached element (idle callback re-arms observePosition)対応中かも @AndreasOhlsson が 21 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
formkit/auto-animate#246 · コメント 1 件 ·
-
`auto-animate/nuxt`: Module causes 'is not wrapped in defineNuxtPlugin' warning since v0.10.0対応中かも @ExEr7um が 106 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
formkit/auto-animate#240 · リアクション 6 件 ·
-
Child added animation is not working in react StrictMode (or in any other case where autoAnimate() is called twice)対応中かも @kaigritun が 262 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
formkit/auto-animate#232 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
formkit/auto-animate#235 · コメント 1 件 · リアクション 1 件 ·
-
Image to Videoオープン
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
formkit/auto-animate#233 ·
formkit/auto-animate の issue をすべて見る
似ている issue
-
b:minor bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
software-mansion/argent#1698 ·
メンテナーはふだん 1 日以内に返信
-
e2e-failure ready-to-code
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
redhat-developer/rhdh-plugin-export-overlays#4624 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
better-auth/better-auth#12132 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
automation missing-model model-sync provider:tinfoil
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
anomalyco/models.dev#12053 ·
メンテナーはふだん 1 日以内に返信
-
o8 next debug build aborts on Glass to Dark appearance switch (nil identifier messaged before check)オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
hurttlocker/o8#5112 ·