Skip to content

fix: 修复协议空间偶现无法找到标签页问题#1903

Merged
MistEO merged 1 commit intov2from
fix/2026040303
Apr 3, 2026
Merged

fix: 修复协议空间偶现无法找到标签页问题#1903
MistEO merged 1 commit intov2from
fix/2026040303

Conversation

@zmdyy0318
Copy link
Copy Markdown
Contributor

@zmdyy0318 zmdyy0318 commented Apr 3, 2026

close #1871

Summary by Sourcery

Bug Fixes:

  • 通过更新 OperationalManual 配置,修复了在 ProtocolSpace 视图中偶发无法定位选项卡的问题。
Original summary in English

Summary by Sourcery

Bug Fixes:

  • Resolve an intermittent issue where tabs in the ProtocolSpace view could not be located by updating the OperationalManual configuration.

Copilot AI review requested due to automatic review settings April 3, 2026 12:29
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

嗨——我已经审查了你的改动,一切看起来都很棒!


Sourcery 对开源项目是免费的——如果你觉得我们的审查有帮助,请考虑分享给其他人 ✨
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进之后的代码审查。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

该 PR 针对 #1871「协议空间高阶培养 I 偶现无法找到入口」的问题,通过调整行动手册内切换标签页时的等待策略,降低界面尚未稳定就进入后续识别/滚动逻辑而导致“直接滑到最底端”的概率。

Changes:

  • 移除进入“索引”标签页后的 post_wait_freezes,减少不必要的等待。
  • 在切换到协议空间对应标签页前新增 pre_wait_freezes,并提高切换后的 post_wait_freezes,让界面更充分稳定后再进行后续识别/滚动。

Comment on lines 41 to 42
],
"action": "Click",
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

这里移除了 post_wait_freezes,但该节点仍会受到 schema 默认 post_delay=200ms 的影响;如果期望点击后立刻进入下一个识别节点,建议显式设置 post_delay(以及必要时 pre_delay)以避免依赖隐式默认值导致时序难以调参。

Suggested change
],
"action": "Click",
],
"pre_delay": 0,
"action": "Click",
"post_delay": 0,

Copilot uses AI. Check for mistakes.
"員養成",
"Operator Progression"
],
"pre_wait_freezes": 200,
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

新增 pre_wait_freezes 后,这个节点在点击前的实际等待会是 pre_wait_freezes + 默认 pre_delay(200ms)。如果这里想用“等待画面静止”来替代盲目延迟,建议同时把 pre_delay 显式设为 0(或显式给出期望值),避免默认值造成额外等待。

Suggested change
"pre_wait_freezes": 200,
"pre_wait_freezes": 200,
"pre_delay": 0,

Copilot uses AI. Check for mistakes.
"ProtocolSpaceOperationalManualFindProtocolSpaceDisableAnchor": "ProtocolSpaceOperationalManualFindProtocolSpaceNormalDisable"
},
"post_wait_freezes": 200,
"post_wait_freezes": 400,
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

post_wait_freezes 从 200 增加到 400 后,点击后的等待将是 post_wait_freezes + 默认 post_delay(200ms)。若只是想把等待时间从 200 调到 400,建议同时显式设置 post_delay=0,避免把默认延迟叠加进去导致实际等待与预期不一致。

Suggested change
"post_wait_freezes": 400,
"post_wait_freezes": 400,
"post_delay": 0,

Copilot uses AI. Check for mistakes.
@MistEO MistEO merged commit bc603ad into v2 Apr 3, 2026
22 checks passed
@MistEO MistEO deleted the fix/2026040303 branch April 3, 2026 14:11
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.

协议空间高阶培养I无法找到入口

3 participants