Skip to content

fix: conditionally enable video skip on double click#990

Open
xiaoxustudio wants to merge 1 commit into
OpenWebGAL:devfrom
xiaoxustudio:fix/playVideo-skipOff-issue
Open

fix: conditionally enable video skip on double click#990
xiaoxustudio wants to merge 1 commit into
OpenWebGAL:devfrom
xiaoxustudio:fix/playVideo-skipOff-issue

Conversation

@xiaoxustudio

Copy link
Copy Markdown
Member

fixes: #985

双击是通过事件直接调用 skipVideo → endPerform(),绕过了场景控制器的判断,设置blockingNext无法阻止。

不过我看之前改脚本这块的逻辑好像大致也是这样的,是改了演出系统的原因才导致失效的嘛...

@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 modifies the video playback logic in playVideo.tsx to conditionally register the double-click event listener for skipping videos. Specifically, the event listener is now only registered if blockingNextFlag is false, preventing users from skipping videos when a block is active. There are no review comments, and I have no feedback to provide.

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.

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