bug: settings overlay blanks the underlying dashboard #306
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
status:blocked
track:api
track:auto
track:core
track:deploy
track:infra
track:ui
type:cleanup
type:docs
type:epic
type:release
type:research
wontfix
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
barrettruth/delta#306
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Parent: #305
Problem
After the settings routing simplification, settings still render with a modal shell, but the previous dashboard route is no longer mounted behind the dialog. Opening settings from queue, kanban, or calendar leaves the blurred backdrop over a blank page instead of over the existing content.
Evidence
src/app/(dashboard)/layout.tsxnow renders only the primarychildrenroute.src/app/(dashboard)/settings/layout.tsxrendersSettingsModalShellas the page content.src/components/settings-modal-shell.tsxportals a modal and backdrop, but there is no preserved prior route underneath.src/lib/settings-navigation.tspreservesreturnTo, but that only controls close behavior; it does not keep the previous dashboard view mounted.Solution
Restore modal-over-current-page behavior without bringing back broad settings complexity.
Preferred implementation:
/settings/*pagesSettingsModalShelldistinguish intercepted modal close behavior from direct settings-page fallbackreturnToquery handling for direct links and hard reloadsAcceptance criteria
/,/?view=queue,/kanban, and/calendar?mode=weekshows the current page behind the backdrop.q,Esc, close button, and browser Back close the overlay and return to the original route including query params./settings/*behavior is deliberate and tested as either a full-page fallback or a documented default background.just lintpass.barrettruth referenced this issue2026-05-11 19:21:02 +00:00