Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

WebKit: removing an animating table row throws in deletePosition()

オープン
#248 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

@thavguard がすでに取り組んでいます。

2026年10月21日 から。

  • #249 @thavguard による — オープン

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
36/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
javascript, typescript, web-dev
領域
frontend, testing

調査の方向性

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 を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

formkit/auto-animate のほかの issue

formkit/auto-animate の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。