bullshit
This commit is contained in:
11
.claude/memory/feedback_fullscreen_modals.md
Normal file
11
.claude/memory/feedback_fullscreen_modals.md
Normal file
@@ -0,0 +1,11 @@
|
||||
---
|
||||
name: Full-screen modals
|
||||
description: App session modals and overlays must cover the full viewport, not just the right panel area of the dashboard
|
||||
type: feedback
|
||||
---
|
||||
|
||||
Modals and app session overlays must be **full screen** — covering the entire viewport including the sidebar/nav. Do NOT constrain them to just the right content panel of the dashboard layout.
|
||||
|
||||
**Why:** The user has corrected this multiple times. Modals that only cover the right panel look wrong and don't provide an immersive app experience.
|
||||
|
||||
**How to apply:** When creating overlays, modals, or app session views, use `position: fixed; inset: 0; z-index: 2400+` to cover the entire screen. The existing AppLauncherOverlay already does this correctly with `class="fixed inset-0 z-[2400]"` — follow that pattern. On mobile it should be truly fullscreen (no padding/margins). On desktop, the glass panel with margins (md:p-10, md:rounded-2xl) is fine.
|
||||
Reference in New Issue
Block a user