Skip to content

fix(dialog): synchronize native modal activation - #95

Open
Banegasn wants to merge 1 commit into
mainfrom
codex/fix-dialog-webkit-stack-stability
Open

fix(dialog): synchronize native modal activation#95
Banegasn wants to merge 1 commit into
mainfrom
codex/fix-dialog-webkit-stack-stability

Conversation

@Banegasn

Copy link
Copy Markdown
Owner

Summary\n- activate the native dialog top-layer state during Lit's post-render open update\n- retain deferred initial focus for slotted component rendering\n- verify stacked dialogs are natively open after their open update\n\n## Verification\n- (Chromium, Firefox, WebKit)\n- \n-

@Banegasn
Banegasn force-pushed the codex/fix-dialog-webkit-stack-stability branch from 266a2e3 to 37ea6e9 Compare July 30, 2026 08:36
@Banegasn

Copy link
Copy Markdown
Owner Author

Acceptance review found a P1 reconnect regression: an open dialog is deactivated/closed natively on disconnect but retains open=true; on reconnect it restores stack/inert state without reopening the native modal because open has not changed. Restore native activation after the rendered reconnect path and add a remove/reappend open-dialog regression asserting native .open/top-layer modal behavior. Implementation has been returned for this focused correction.

@Banegasn
Banegasn force-pushed the codex/fix-dialog-webkit-stack-stability branch 2 times, most recently from 3ee6bf6 to 884c551 Compare July 30, 2026 08:47
@Banegasn
Banegasn force-pushed the codex/fix-dialog-webkit-stack-stability branch from 884c551 to c8dc99f Compare July 30, 2026 08:54
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