Skip to content

Commit d0ddc43

Browse files
test: pin app-footer mobile/desktop visibility; trim comment
Add a Playwright regression test asserting .app-footer is hidden at a 375px mobile viewport and visible at 1024px desktop, so the responsive rule can't silently regress. Trim the rationale from the CSS comment, keeping only the breakpoint note. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 8909e4f commit d0ddc43

2 files changed

Lines changed: 20 additions & 2 deletions

File tree

src/routes/(app)/+layout.svelte

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,7 @@
7575
}
7676
}
7777
78-
/* Hide the footer on mobile / small viewports — it costs too much screen
79-
real estate there. Mirrors the Header's 768px desktop breakpoint. */
78+
/* Match Header's 768px desktop breakpoint. */
8079
@media only screen and (max-width: 767.98px) {
8180
.app-footer {
8281
display: none;
Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
import { expect, test } from '@playwright/test'
2+
3+
// Regression test for hiding the compact app footer on mobile viewports
4+
// (issue encryption4all/postguard-website#279). The footer lives in the (app)
5+
// layout and is hidden below Header's 768px desktop breakpoint, where it cost
6+
// too much screen real estate. It must stay visible on desktop.
7+
const FOOTER = '.app-footer'
8+
9+
test('app footer is hidden on a mobile viewport', async ({ page }) => {
10+
await page.setViewportSize({ width: 375, height: 812 })
11+
await page.goto('/fileshare/')
12+
await expect(page.locator(FOOTER)).toBeHidden()
13+
})
14+
15+
test('app footer is visible on a desktop viewport', async ({ page }) => {
16+
await page.setViewportSize({ width: 1024, height: 768 })
17+
await page.goto('/fileshare/')
18+
await expect(page.locator(FOOTER)).toBeVisible()
19+
})

0 commit comments

Comments
 (0)