Conversation
There was a problem hiding this comment.
嗨——我已经审查了你的改动,一切看起来都很棒!
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进之后的代码审查。
Original comment in English
Hey - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Pull request overview
该 PR 针对 #1871「协议空间高阶培养 I 偶现无法找到入口」的问题,通过调整行动手册内切换标签页时的等待策略,降低界面尚未稳定就进入后续识别/滚动逻辑而导致“直接滑到最底端”的概率。
Changes:
- 移除进入“索引”标签页后的
post_wait_freezes,减少不必要的等待。 - 在切换到协议空间对应标签页前新增
pre_wait_freezes,并提高切换后的post_wait_freezes,让界面更充分稳定后再进行后续识别/滚动。
| ], | ||
| "action": "Click", |
There was a problem hiding this comment.
这里移除了 post_wait_freezes,但该节点仍会受到 schema 默认 post_delay=200ms 的影响;如果期望点击后立刻进入下一个识别节点,建议显式设置 post_delay(以及必要时 pre_delay)以避免依赖隐式默认值导致时序难以调参。
| ], | |
| "action": "Click", | |
| ], | |
| "pre_delay": 0, | |
| "action": "Click", | |
| "post_delay": 0, |
| "員養成", | ||
| "Operator Progression" | ||
| ], | ||
| "pre_wait_freezes": 200, |
There was a problem hiding this comment.
新增 pre_wait_freezes 后,这个节点在点击前的实际等待会是 pre_wait_freezes + 默认 pre_delay(200ms)。如果这里想用“等待画面静止”来替代盲目延迟,建议同时把 pre_delay 显式设为 0(或显式给出期望值),避免默认值造成额外等待。
| "pre_wait_freezes": 200, | |
| "pre_wait_freezes": 200, | |
| "pre_delay": 0, |
| "ProtocolSpaceOperationalManualFindProtocolSpaceDisableAnchor": "ProtocolSpaceOperationalManualFindProtocolSpaceNormalDisable" | ||
| }, | ||
| "post_wait_freezes": 200, | ||
| "post_wait_freezes": 400, |
There was a problem hiding this comment.
post_wait_freezes 从 200 增加到 400 后,点击后的等待将是 post_wait_freezes + 默认 post_delay(200ms)。若只是想把等待时间从 200 调到 400,建议同时显式设置 post_delay=0,避免把默认延迟叠加进去导致实际等待与预期不一致。
| "post_wait_freezes": 400, | |
| "post_wait_freezes": 400, | |
| "post_delay": 0, |
close #1871
Summary by Sourcery
Bug Fixes:
Original summary in English
Summary by Sourcery
Bug Fixes: