Skip to content

[Frontend][Paper] Card inspector follow-ups from the #2097/#2100 review rounds (P2 batch) #2129

Description

@Chris0Jeky

Unaddressed Codex P2s from merged PRs #2097/#2100, batched (2026-08-26 sweep):

  1. Desktop @media(min-width:768px) .card-modal-viewport rule (equal specificity) overrides inspector top/height with inset:0/height:auto.
  2. Density control in the non-wrapping flex:none action row overflows a 390px viewport, pushing header controls off-screen.
  3. After a confirmed A->B card switch, CardModal keeps card A as previouslyFocusedElement; closing B returns focus to A (possibly off-screen).
  4. On desktop-inspector -> modal transition (resize/orientation), the isOpen watcher does not rerun: focus stays behind the overlay, Tab trap not engaged.
  5. Fire-and-forget fetchCardComments has no rejection handler -> unhandled rejection on failed loads.
  6. beforeunload is registered for the whole Paper board visit, not only while dirty -> disables Firefox bfcache on clean boards (full reload).

Refs #2097 #2100.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Priority IIIValuable but unscheduled: residuals, tech-debt, performance, depth work.frontendPrimary implementation impact in Vue/TypeScript UI and client runtime.uxDiscoverability, accessibility, interaction model, and user workflow quality.

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions