Skip to content

feat(settings): open web settings in an in-page dialog - #354

Open
AAEE86 wants to merge 2 commits into
xintaofei:mainfrom
AAEE86:main
Open

feat(settings): open web settings in an in-page dialog#354
AAEE86 wants to merge 2 commits into
xintaofei:mainfrom
AAEE86:main

Conversation

@AAEE86

@AAEE86 AAEE86 commented Jul 17, 2026

Copy link
Copy Markdown
  • replace the web settings window with a responsive modal
  • reuse existing settings routes through a same-origin iframe
  • support loading state, close button, overlay and Escape dismissal
  • add tests for dialog events and interactions

AAEE86 added 2 commits July 17, 2026 17:42
- replace the web settings window with a responsive modal
- reuse existing settings routes through a same-origin iframe
- support loading state, close button, overlay and Escape dismissal
- add tests for dialog events and interactions
- replace the browser commit window with a responsive embedded modal
- close the modal after commit or cancel and support Escape dismissal
- preserve the desktop Tauri window behavior
- add event and dialog regression tests
xintaofei added a commit that referenced this pull request Aug 2, 2026
1.1.9 (#354) coalesces streamed plan snapshots into one `plan_update`
every 150ms and flushes them at item, turn, permission, and disposal
boundaries. That path runs only for clients advertising
`clientCapabilities.plan`, which codeg does not: sacp 11.0.0's schema
has neither the capability nor the session-update variant, so Codex
plans keep arriving as `agent_message_chunk`s and nothing here changes.
Dependencies are identical to 1.1.8's and there is still no
`engines.node`, so the Node 20.0.0 floor stands.

Refresh the doc comments that name the pinned release, including the
still-absent `promptRequired` steering opt-in that keeps Codex on the
MCP pull channel, re-verified against the 1.1.9 tarball.
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