Skip to content

fix: 修复 changeFigure -next 后 setTransform 继承 transform 错误#994

Open
A-kirami wants to merge 2 commits into
OpenWebGAL:devfrom
A-kirami:fix/fast-preview-next-transform-settling
Open

fix: 修复 changeFigure -next 后 setTransform 继承 transform 错误#994
A-kirami wants to merge 2 commits into
OpenWebGAL:devfrom
A-kirami:fix/fast-preview-next-transform-settling

Conversation

@A-kirami

Copy link
Copy Markdown
Contributor

背景

changeFigure -transform -next 后紧接 setTransform 时,fast preview 会把 changeFigure 的 pending enter 终态延后到下一轮 forward() 才结算,导致它覆盖后续 setTransform 已写入的 transform。

#992

改动

  • 在 fast preview 的 -next 递归执行下一句前,结算当前句 pending non-hold perform 的演算终态。
  • 保持普通播放路径不变。

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces the settleFastPreviewPendingStateBeforeNext function to settle pending non-hold performance states during fast preview. The reviewer identified a potential state inconsistency issue where the backlog is saved before these states are settled, and suggested moving the settlement function call before saveBacklogIfNeeded.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread packages/webgal/src/Core/controller/gamePlay/scriptExecutor.ts Outdated
@A-kirami A-kirami changed the base branch from main to dev June 25, 2026 23:02
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