Notifications: the inbox panel opens mostly off screen since the bell moved to the sidebar #130

Closed
opened 2026-09-25 22:17:36 +00:00 by kayg · 1 comment
Owner

Severity: high (the inbox is unusable on every desktop width)

Problem

The inbox panel lines its right edge up with the bell's pill group. Since the bell moved into the sidebar header (left side), the 400px panel started about 280px left of the viewport: /notifications and the bell showed only the panel's right edge ("Get them on this device" at x −167). 768, 1024, 1440 and 1920 all fail. /home/kayg/Developer/calternal/target/breakit/run1/notifications_1440-paper.png.

Expected

The panel opens fully on screen under the bell.

Status

Fixed on job/breakit-fixes in 93d6b06: inboxRight() (lib/notifications/model.ts) aligns left edges for a bell in the left half and clamps the panel 12px inside the viewport; unit tests in model.test.ts. Check: /home/kayg/Developer/calternal/target/breakit/verify/v-inbox-1440.png. Not merged. Note for job/headers: when the bell moves again, the panel follows automatically.

Found by the break-it sweep (#117). Re-run: cd apps/web && bun run build && bun e2e/breakit.mjs --keep <dir> (script on branch job/breakit-fixes). Screenshots: /home/kayg/Developer/calternal/target/breakit (run 1 in run1/, fix checks in verify/).

**Severity:** high (the inbox is unusable on every desktop width) ## Problem The inbox panel lines its right edge up with the bell's pill group. Since the bell moved into the sidebar header (left side), the 400px panel started about 280px left of the viewport: `/notifications` and the bell showed only the panel's right edge ("Get them on this device" at x −167). 768, 1024, 1440 and 1920 all fail. `/home/kayg/Developer/calternal/target/breakit/run1/notifications_1440-paper.png`. ## Expected The panel opens fully on screen under the bell. ## Status Fixed on `job/breakit-fixes` in 93d6b06: `inboxRight()` (lib/notifications/model.ts) aligns left edges for a bell in the left half and clamps the panel 12px inside the viewport; unit tests in model.test.ts. Check: `/home/kayg/Developer/calternal/target/breakit/verify/v-inbox-1440.png`. Not merged. Note for job/headers: when the bell moves again, the panel follows automatically. Found by the break-it sweep (#117). Re-run: `cd apps/web && bun run build && bun e2e/breakit.mjs --keep <dir>` (script on branch `job/breakit-fixes`). Screenshots: `/home/kayg/Developer/calternal/target/breakit` (run 1 in `run1/`, fix checks in `verify/`).
Author
Owner

Fixed. The Notifications E2E passed. It checks that the desktop inbox stays inside the viewport while the bell is in the sidebar, and that /notifications opens the inbox and returns to the prior calendar route when closed.

Fixed. The Notifications E2E passed. It checks that the desktop inbox stays inside the viewport while the bell is in the sidebar, and that `/notifications` opens the inbox and returns to the prior calendar route when closed.
kayg closed this issue 2026-09-26 12:51:56 +00:00
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kayg/calternal#130
No description provided.