Skip to content

fix: resolve scroll overflow on mock panels#3

Merged
yepzdk merged 1 commit intomainfrom
feature/fix-mock-panel-scroll
Apr 20, 2026
Merged

fix: resolve scroll overflow on mock panels#3
yepzdk merged 1 commit intomainfrom
feature/fix-mock-panel-scroll

Conversation

@yepzdk
Copy link
Copy Markdown
Contributor

@yepzdk yepzdk commented Apr 20, 2026

Summary

  • Fixed scroll not working on mock panels (Process Map, Dashboard, Operations, etc.)
  • Root cause: body used min-height: 100vh instead of height: 100vh, allowing the flex layout to grow beyond the viewport so overflow-y: auto on panels never activated
  • Changed body to fixed viewport height, added min-height: 0 on flex ancestors, removed forced min-height: 400px on process canvas
  • Most panels now use display: block; only Tasks and AI Assistant use display: flex (needed for kanban board and chat stretching)

Test plan

  • Process Map → scroll the process list table
  • Process Map → click Parking Complaint → scroll down to see version history
  • Dashboard → scroll to see full chart section on small viewports
  • Operations → scroll to see alerts table
  • AI Assistant → chat messages scroll within the chat container
  • Agents → detail view scrolls

🤖 Generated with Claude Code

The body used min-height: 100vh which allowed the flex layout to grow
beyond the viewport — so overflow-y: auto on panels never activated.

- body: min-height: 100vh → height: 100vh (constrain to viewport)
- .main: add min-height: 0 (allow flex shrink)
- .topbar: add flex-shrink: 0 (prevent topbar from collapsing)
- .panel: add min-height: 0, use display: block for most panels
- .process-canvas: remove min-height: 400px (forced oversized content)

Co-authored-by: Claude <noreply@anthropic.com>
@yepzdk yepzdk merged commit 1cddd33 into main Apr 20, 2026
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