Skip to content

fix(den-web): contain and dismiss workspace switcher - #3549

Open
reachjalil wants to merge 2 commits into
devfrom
codex/fix-workspace-switcher-popover
Open

fix(den-web): contain and dismiss workspace switcher#3549
reachjalil wants to merge 2 commits into
devfrom
codex/fix-workspace-switcher-popover

Conversation

@reachjalil

@reachjalil reachjalil commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • keep the Den Web workspace switcher within the viewport and prevent long account text from widening its grid
  • dismiss the switcher on outside pointer interaction
  • support Escape dismissal with focus returning to the trigger
  • add focused regression coverage for the menu bounds and dismissal wiring

Root cause

The fixed-width switcher used an intrinsic grid column, so a long unbroken email could widen its contents beyond the panel. The hand-rolled popover also had no document-level outside-interaction handler.

User impact

Long account emails stay contained inside the switcher, and clicking elsewhere closes the open panel.

Validation

  • bun test ee/apps/den-web/tests/org-dashboard-shell-layout.test.ts — 2 passed
  • corepack pnpm --filter @openwork-ee/den-web typecheck — passed
  • git diff --check origin/dev...HEAD — passed

Base: 3620005c3792d20deec993e173ec3302cbe12dbb
Head: 29d4f66f3beb24c321802f57995c5f9ba6d1499e

Deferred verification

The environment-backed Den Web acceptance journey and evidence tape were not run in this quick working-deliverable pass.

@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview Aug 5, 2026 2:21pm
openwork-den Ready Ready Preview Aug 5, 2026 2:21pm
openwork-den-worker-proxy Ready Ready Preview Aug 5, 2026 2:21pm
openwork-landing Ready Ready Preview, v0 Aug 5, 2026 2:21pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
openwork-diagnostics Skipped Skipped Aug 5, 2026 2:21pm

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant