Skip to content

🐛 fix: 開いて畳んだ面のドラッグプレビューを修正 - #81

Open
yutteee wants to merge 1 commit into
mainfrom
codex/fix-connected-fold-preview
Open

yutteee wants to merge 1 commit into
mainfrom
codex/fix-connected-fold-preview

Conversation

@yutteee

@yutteee yutteee commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

「開いて畳む」の後に角をドラッグすると、プレビューは左側の三角形だけを表示する一方、確定後はつながった左右の面が一緒に折れていました。プレビューが候補の先頭1面だけを描き、ドロップ時の接続・破れ判定を使っていなかったことが原因です。

通常の折りが成立する最少枚数を applyFoldStep で求め、その操作で動く面をすべてプレビューします。表裏それぞれの視点と元の描画高さを保ち、複数メッシュを更新・終了時にまとめて破棄します。通常の折りが成立しない位置には不成立の形を表示せず、特殊折りの選択肢は従来どおりドロップ時に判定します。

確認内容:

  • Storybookで「対角線折り2回 → 開いて畳む → 右上の角を内側へドラッグ」を操作し、報告画像と同じ不具合を再現。修正後は左右を含む三角形全体の予告、折り途中、確定後の形を目視確認。
  • 同じ操作は1面では不成立・2面では成立する回帰テストを追加。修正前に右側の面が欠けるアサーションが失敗することを確認。
  • 折り紙機能: 39ファイル / 231テスト通過。表裏、既存の折り目でのめくり、独立して折れる面、紙が破れる操作、複数メッシュの破棄を確認。
  • 変更したTypeScriptファイルのESLint・差分チェック・コミットメッセージ検証に通過。
  • プレビュー計算100回のローカル計測: 6面で中央値0.06ms、40面で中央値1.87ms / 95パーセンタイル3.25ms(計算のみ、描画を除く)。
  • 全体の型チェックは既存の packages/design-system のエラー4件で失敗: react-aria-components が解決できない2件と、それに伴う暗黙anyの2件。今回の変更箇所での型エラーはありません。

@vercel

vercel Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ori-cube Ready Ready Preview Sep 5, 2026 2:24pm UTC
ori-cube-storybook Ready Ready Preview Sep 5, 2026 2:24pm UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant