Skip to content

Commit 585a40b

Browse files
committed
fix(screenshot): uses client-side nav to switch density mode
1 parent 75539ac commit 585a40b

File tree

3 files changed

+19
-4
lines changed

3 files changed

+19
-4
lines changed
-4.87 KB
Loading

docs/dashboard-screenshot.png

-99 Bytes
Loading

e2e/capture-screenshot.spec.ts

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -473,10 +473,25 @@ test("capture dashboard screenshot", async ({ page }) => {
473473

474474
await page.screenshot({ path: "docs/dashboard-screenshot.png" });
475475

476-
// Switch to compact density and capture a second screenshot
477-
await page.evaluate(() => {
478-
document.documentElement.dataset.density = "compact";
479-
});
476+
// Switch to compact density: update the config in localStorage and reload
477+
// so SolidJS reactive state (isCompact memos, <Show> guards) picks it up.
478+
// Switch to compact density via the settings UI (client-side nav to preserve store state)
479+
await page.getByRole("link", { name: "Settings" }).click();
480+
await page.getByRole("button", { name: /view density: compact/i }).waitFor();
481+
await page.getByRole("button", { name: /view density: compact/i }).click();
482+
await page.getByRole("button", { name: /view density: compact/i, pressed: true }).waitFor();
483+
484+
// Navigate back to dashboard via the back link (client-side, no full reload)
485+
await page.getByRole("link", { name: "Back to dashboard" }).click();
486+
await page.getByRole("tablist").waitFor();
487+
await page.getByRole("tab", { name: /pull requests/i }).click();
488+
await page.getByRole("tab", { name: /pull requests/i, selected: true }).waitFor();
489+
await page.getByText("acme-corp/web-platform").first().waitFor();
490+
const compactRepoBtn = page.getByRole("button", { expanded: false }).filter({ hasText: "acme-corp/web-platform" });
491+
if (await compactRepoBtn.isVisible()) {
492+
await compactRepoBtn.click();
493+
await page.getByRole("button", { expanded: true }).filter({ hasText: "acme-corp/web-platform" }).waitFor();
494+
}
480495

481496
await page.screenshot({ path: "docs/dashboard-screenshot-compact.png" });
482497
});

0 commit comments

Comments
 (0)