Skip to content

Fix stuck on overlay#316

Merged
CJackHwang merged 1 commit intoCJackHwang:mainfrom
catsimple:fix_stuck_on_overlay
Apr 17, 2026
Merged

Fix stuck on overlay#316
CJackHwang merged 1 commit intoCJackHwang:mainfrom
catsimple:fix_stuck_on_overlay

Conversation

@catsimple
Copy link
Copy Markdown
Contributor

16:39:12.140 ERR SYS d25rwck Error setting Thinking Level: Locator.click: Timeout 3000ms exceeded.
Call log:
- waiting for locator("[role=\"combobox\"][aria-label=\"Thinking Level\"], mat-select[aria-label=\"Thinking Level\"], [role=\"combobox\"][aria-label=\"Thinking level\"], mat-select[aria-label=\"Thinking level\"]")
- locator resolved to <mat-select tabindex="0" role="combobox" id="mat-select-0" aria-invalid="false" aria-expanded="false" aria-required="false" aria-disabled="false" class="mat-mdc-select" aria-haspopup="listbox" aria-label="Thinking Level" _ngcontent-ng-c2997479666="">…</mat-select>
- attempting click action
2 × waiting for element to be visible, enabled and stable
- element is visible, enabled and stable
- scrolling into view if needed
- done scrolling
- <div class="cdk-overlay-backdrop cdk-overlay-transparent-backdrop cdk-overlay-backdrop-showing"></div> from <div class="cdk-overlay-container">…</div> subtree intercepts pointer events
- retrying click action
- waiting 20ms

在上一个提交中,引入了打开View more actions的操作,执行这个操作后,在旧代码中无法正确识别当前temporary chat状态,或是可能识别到了temporary chat状态是已打开的状态,后续发送Escape因为某些原因失败,导致View more actions这个overlay一直处于打开状态,进而无法进行其它操作如上传文件/图片、设置thinking level等

该提交执行以下修复:

  1. temporary chat状态判断前置,先判断当前temporary chat状态再选择是否要打开View more actions
  2. 打开菜单后,无论后续判断或操作如何,最后都执行_close_menu_if_needed()尝试关闭overlay,避免卡住其它后续操作

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@CJackHwang CJackHwang merged commit fd1da7b into CJackHwang:main Apr 17, 2026
4 of 6 checks passed
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.

2 participants