Skip to content

Commit c5b1972

Browse files
PureWeenCopilot
andcommitted
Fix popup: use JS dialog outside Blazor DOM tree
Previous fix removed the JS DOM portal but kept the dialog in Blazor's render tree. The overlay showed but the dialog was clipped/invisible due to ancestor overflow constraints that position:fixed alone can't escape in WebView with scoped CSS. Adopt the approach from PR #226: render the close-confirmation dialog entirely via JS (window.showCloseSessionDialog), creating DOM elements imperatively and appending to document.body. Blazor never tracks these elements, so no render batch desync on open/close/delete. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent d8978f3 commit c5b1972

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

PolyPilot/Components/Layout/SessionListItem.razor

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
@inject RepoManager RepoManager
66
@inject IJSRuntime JS
77

8-
98
@{
109
var isPinned = Meta?.IsPinned ?? false;
1110
var sessionDir = GetSessionDirectory();

0 commit comments

Comments
 (0)