Skip to content

Commit 72d6d83

Browse files
authored
fix(app): unmount hidden session panes (anomalyco#35628)
1 parent 3a7117b commit 72d6d83

4 files changed

Lines changed: 286 additions & 306 deletions

File tree

packages/app/e2e/regression/review-terminal-stacked.spec.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -168,8 +168,8 @@ test("keeps the review tree and terminal sized when both panels are open", async
168168
await expectTree(page, 2_773, "action.yml")
169169

170170
await page.getByRole("button", { name: "Toggle file tree" }).click()
171-
await expect(page.locator('[data-slot="session-review-v2-sidebar"]')).toHaveAttribute("aria-hidden", "true")
172-
await expect(page.locator('#review-panel [data-component="file-tree-v2"]')).toHaveCount(1)
171+
await expect(page.locator('[data-slot="session-review-v2-sidebar"]')).toHaveCount(0)
172+
await expect(page.locator('#review-panel [data-component="file-tree-v2"]')).toHaveCount(0)
173173
await page.getByRole("button", { name: "Toggle file tree" }).click()
174174
await expectTree(page, 2_773, "action.yml")
175175

@@ -181,8 +181,7 @@ test("keeps the review tree and terminal sized when both panels are open", async
181181
await expectTree(page, 2_773, "action.yml")
182182

183183
await page.getByRole("button", { name: "Toggle review" }).click()
184-
await expect(page.locator("#review-panel")).toHaveAttribute("aria-hidden", "true")
185-
await expect(page.locator('#review-panel [data-component="file-tree-v2"]')).toHaveCount(1)
184+
await expect(page.locator("#review-panel")).toHaveCount(0)
186185
await page.getByRole("button", { name: "Toggle review" }).click()
187186
await expectTree(page, 2_773, "action.yml")
188187
await page.setViewportSize({ width: 1_000, height: 700 })

packages/app/src/pages/session.tsx

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2198,7 +2198,7 @@ export default function Page() {
21982198
</Show>
21992199
</div>
22002200

2201-
<Show when={!newSessionDesign()}>
2201+
<Show when={!newSessionDesign() && desktopSidePanelOpen()}>
22022202
<SessionSidePanel
22032203
canReview={canReview}
22042204
diffs={reviewDiffs}
@@ -2217,13 +2217,8 @@ export default function Page() {
22172217
<Show when={newSessionDesign()}>
22182218
<Show when={isDesktop() ? desktopV2PanelLayout().visible : terminalOpen()}>
22192219
<div class="min-w-0 h-full flex flex-1 flex-col">
2220-
<Show when={isDesktop()}>
2221-
<div
2222-
classList={{
2223-
"min-h-0 flex-1": desktopV2ReviewOpen() || desktopFileTreeOpen(),
2224-
"size-0 shrink-0 overflow-hidden": !(desktopV2ReviewOpen() || desktopFileTreeOpen()),
2225-
}}
2226-
>
2220+
<Show when={isDesktop() && (desktopV2ReviewOpen() || desktopFileTreeOpen())}>
2221+
<div class="min-h-0 flex-1">
22272222
<SessionSidePanel
22282223
canReview={canReview}
22292224
diffs={reviewDiffs}

0 commit comments

Comments
 (0)