rstillwell/deckhearth
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

fix(responsive): prevent horizontal overflow on mobile #159

Merged
rstillwell merged 2 commits from fix/mobile-overflow into main 2026-09-11 21:06:53 -04:00
Conversation 0 Commits 2 Files changed 9 +17 -15
rstillwell commented 2026-09-11 21:06:22 -04:00
Owner
Copy link
  • styles/globals.css: add overflow-x: hidden + max-width: 100vw to body (global safety net)
  • components/CollectionPageView.js: flex-wrap filter row, responsive search width (w-full sm:w-64)
  • pages/my-cards.js: replace min-w-[200px] with w-full sm:min-w-[200px] on filter search
  • pages/designer.js: remove shrink-0 from live preview sidebar to allow flex collapse
  • components/DeckDetailView.js: add flex-wrap + gap to header flex row
  • components/CardItem.js: remove whitespace-nowrap from expanding add buttons (3 instances)
  • components/DashboardRecentActivity.js: whitespace-nowrap → whitespace-normal on timestamp
  • components/CollaboratorFacepile.js: tooltips use whitespace-normal + break-words + max-w
  • components/MobileNavigation.js: tooltip whitespace-nowrap → whitespace-normal

Fixes horizontal scroll / layout breakage on mobile (<375px) across the site.

Summary

Convoy + Brief

  • Convoy: .convoys/<slug>.md
  • Brief: .convoys/<slug>/brief-<N>-...md

Acceptance criteria

  • No scope expansion (only files listed in the brief's files: were edited)

Test plan

Pipeline gates

  • CI: lint, types, build, unit tests
  • Forbidden patterns / npm audit (if installed)
  • Visual diff (if UI change)
  • A11y audit (if UI change)
  • Design-system audit (if UI change)
  • Security audit (if code change)
  • Reviewer report
  • Smoke on staging (after merge to develop)

Security notes

Notes for reviewer

- styles/globals.css: add overflow-x: hidden + max-width: 100vw to body (global safety net) - components/CollectionPageView.js: flex-wrap filter row, responsive search width (w-full sm:w-64) - pages/my-cards.js: replace min-w-[200px] with w-full sm:min-w-[200px] on filter search - pages/designer.js: remove shrink-0 from live preview sidebar to allow flex collapse - components/DeckDetailView.js: add flex-wrap + gap to header flex row - components/CardItem.js: remove whitespace-nowrap from expanding add buttons (3 instances) - components/DashboardRecentActivity.js: whitespace-nowrap → whitespace-normal on timestamp - components/CollaboratorFacepile.js: tooltips use whitespace-normal + break-words + max-w - components/MobileNavigation.js: tooltip whitespace-nowrap → whitespace-normal Fixes horizontal scroll / layout breakage on mobile (<375px) across the site. <!-- PR title for convoy work: convoy: <short description> (pipeline: convoy=<slug>, brief=<N>) skip: <comma-separated flags or empty> Skip flags (Conductor sets these — do not edit by hand): ia, ux, arch, test, review, security, visual, a11y, design, smoke, qa, docs, flag Never skip: plan-approval, pr-merge, prod-promote When convoy-metrics-gate.yml is installed: convoy PRs must add rows to .convoys/.metrics.jsonl. Run scripts/log-convoy-event.sh after each role, or add skip-metrics label with reason in the PR body. --> ## Summary <!-- 2-3 bullets: what changed and why. User-facing language preferred. --> ## Convoy + Brief - Convoy: `.convoys/<slug>.md` - Brief: `.convoys/<slug>/brief-<N>-...md` ## Acceptance criteria <!-- Copy from the brief; check off as you complete. --> - [ ] - [ ] - [ ] No scope expansion (only files listed in the brief's `files:` were edited) ## Test plan <!-- What was tested, how, and what wasn't tested with rationale. --> ## Pipeline gates <!-- Filled in by CI / role-reviewer. Don't edit. --> - [ ] CI: lint, types, build, unit tests - [ ] Forbidden patterns / npm audit (if installed) - [ ] Visual diff (if UI change) - [ ] A11y audit (if UI change) - [ ] Design-system audit (if UI change) - [ ] Security audit (if code change) - [ ] Reviewer report - [ ] Smoke on staging (after merge to develop) ## Security notes <!-- Auth, permissions, or data-handling changes? Link security audit findings. --> ## Notes for reviewer <!-- Anything unusual, intentional trade-offs, or follow-ups. -->
rstillwell added 1 commit 2026-09-11 21:06:23 -04:00
fix(responsive): prevent horizontal overflow on mobile
Some checks failed
Visual diff / Should run? (pull_request) Has been cancelled
Details
CI / Lint (pull_request) Has been cancelled
Details
CI / Schema map up to date (pull_request) Has been cancelled
Details
CI / Forbidden patterns (9 checks) (pull_request) Has been cancelled
Details
CI / Migrations apply (node-pg-migrate) (pull_request) Has been cancelled
Details
CI / Unit tests (vitest) (pull_request) Has been cancelled
Details
Convoy metrics gate / Require role-event telemetry on convoy PRs (pull_request) Has been cancelled
Details
PR Health rollup / Aggregate gate status (pull_request) Has been cancelled
Details
Preview smoke / Should run? (pull_request) Has been cancelled
Details
Visual diff / Screenshot diff (pull_request) Has been cancelled
Details
Preview smoke / Playwright smoke (pull_request) Has been cancelled
Details
e6869c82cc 
- styles/globals.css: add overflow-x: hidden + max-width: 100vw to body (global safety net)
- components/CollectionPageView.js: flex-wrap filter row, responsive search width (w-full sm:w-64)
- pages/my-cards.js: replace min-w-[200px] with w-full sm:min-w-[200px] on filter search
- pages/designer.js: remove shrink-0 from live preview sidebar to allow flex collapse
- components/DeckDetailView.js: add flex-wrap + gap to header flex row
- components/CardItem.js: remove whitespace-nowrap from expanding add buttons (3 instances)
- components/DashboardRecentActivity.js: whitespace-nowrap → whitespace-normal on timestamp
- components/CollaboratorFacepile.js: tooltips use whitespace-normal + break-words + max-w
- components/MobileNavigation.js: tooltip whitespace-nowrap → whitespace-normal

Fixes horizontal scroll / layout breakage on mobile (<375px) across the site.
rstillwell added 1 commit 2026-09-11 21:06:42 -04:00
Merge branch 'main' into fix/mobile-overflow
Some checks failed
CI / Lint (pull_request) Has been cancelled
Details
CI / Schema map up to date (pull_request) Has been cancelled
Details
CI / Forbidden patterns (9 checks) (pull_request) Has been cancelled
Details
CI / Migrations apply (node-pg-migrate) (pull_request) Has been cancelled
Details
CI / Unit tests (vitest) (pull_request) Has been cancelled
Details
Convoy metrics gate / Require role-event telemetry on convoy PRs (pull_request) Has been cancelled
Details
PR Health rollup / Aggregate gate status (pull_request) Has been cancelled
Details
Preview smoke / Should run? (pull_request) Has been cancelled
Details
Visual diff / Should run? (pull_request) Has been cancelled
Details
Preview smoke / Playwright smoke (pull_request) Has been cancelled
Details
Visual diff / Screenshot diff (pull_request) Has been cancelled
Details
cfb5ab5649
rstillwell merged commit 5f64635ce8 into main 2026-09-11 21:06:53 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
agent-context-drift

   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
agent-context-drift
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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: rstillwell/deckhearth#159
No description provided.
Delete branch "fix/mobile-overflow"

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?