Skip to content

Fix admin table columns hidden on overflow instead of scrollable - #202

Merged
calebyhan merged 1 commit into
mainfrom
#199/admin-table-scrollbar
Aug 21, 2026
Merged

Fix admin table columns hidden on overflow instead of scrollable#202
calebyhan merged 1 commit into
mainfrom
#199/admin-table-scrollbar

Conversation

@calebyhan

Copy link
Copy Markdown
Contributor

Summary

  • DataTable.tsx clipped table overflow with overflow-hidden instead of allowing horizontal scroll, and cells use whitespace-nowrap, so long field values pushed trailing columns (type/status/actions) off-screen with no way to reach them.
  • Switched the container to overflow-x-auto (dropping overflow-hidden) and added min-w-max to the table so it doesn't compress below content width.
  • This is a shared component used by all 9 admin tables (events, senators, legislation, news, districts, accounts, leadership, static-pages, staff), so the fix applies everywhere at once.

Fixes #199

Test plan

  • tsc --noEmit passes
  • Frontend build passes
  • Manually verify the events table (and a couple others) scroll horizontally on a narrow viewport instead of hiding the actions column

The table container used overflow-hidden with no-wrap cells, so long
field values pushed trailing columns (type/status/actions) off-screen
with no way to reach them. Switch to overflow-x-auto so the table
scrolls horizontally instead of clipping content.

Fixes #199
@github-actions

Copy link
Copy Markdown

Test Results

573 tests  ±0   573 ✅ ±0   49s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit f3c1047. ± Comparison against base commit 9526ecc.

@calebyhan
calebyhan marked this pull request as ready for review August 21, 2026 01:10

@MasonMines2006 MasonMines2006 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@calebyhan
calebyhan merged commit 3c3f1e1 into main Aug 21, 2026
3 checks passed
@calebyhan
calebyhan deleted the #199/admin-table-scrollbar branch August 21, 2026 14:31
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.

Missing scrollbar on admin dashboard

2 participants