Skip to content

The report's wide charts scroll instead of escaping - #32

Merged
AllTerrainDeveloper merged 1 commit into
mainfrom
fix/analytics-overflow
Aug 21, 2026
Merged

AllTerrainDeveloper merged 1 commit into
mainfrom
fix/analytics-overflow

Conversation

@AllTerrainDeveloper

Copy link
Copy Markdown
Owner

Spotted on the QA site: in a narrow analytics window the cross-tab's bars painted straight through the panel border and off the edge of the window, and the new 40-bucket hours histogram could do the same.

Why

  • A table never shrinks below its content, and its headers were white-space: nowrap whole questions — six metric columns demanded every question laid end to end.
  • A histogram column never shrinks below its count label, and "Hours a week it took" has ~36 buckets.

Fix

  • A shared .atfa-scroll wrapper (overflow-x: auto) around the cross-tab and every histogram: the panel keeps its edge, the content keeps its size, and the strip between them scrolls.
  • Cross-tab headers now wrap over a min-inline-size: 140px metric column instead of nowrap, so the table stays readable long before the scroll has to take over.

Verified on 8889

Squeezed the analytics window narrow and measured every panel: zero elements past any panel edge (was: bars bleeding through), the cross-tab scrolls 188px to its hidden columns, and the hours histogram carries its 102px of overflow inside its own scroller.

353 vitest green, build deployed.

🤖 Generated with Claude Code

https://claude.ai/code/session_0119U4sRRWGcTQdwYreTwpAp

A table never shrinks below its content and a histogram column never
shrinks below its count label, so in a narrow window the cross-tab and
the hours histogram painted straight through the panel border. Both now
live in a sideways-scrolling strip, and the cross-tab's question headers
wrap over a fixed column width instead of demanding every question laid
end to end.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0119U4sRRWGcTQdwYreTwpAp
@AllTerrainDeveloper
AllTerrainDeveloper merged commit f7d319a into main Aug 21, 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