fix(tables): use overflow-clip on header text to allow horizontal scrolling#3722
Conversation
PR SummaryLow Risk Overview Written by Cursor Bugbot for commit fefe8c4. Configure here. |
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryThis PR fixes a horizontal scroll issue in the tables view by replacing Tailwind's Key changes:
Confidence Score: 5/5
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User scrolls horizontally over table header] --> B{Header text span overflow style}
B -->|Before: overflow: hidden| C[Span creates scroll container / BFC]
C --> D[Scroll event consumed by span]
D --> E[❌ Parent table does NOT scroll]
B -->|After: overflow: clip| F[Span clips content, NO scroll container]
F --> G[Scroll event bubbles up normally]
G --> H[✅ Parent table scrolls horizontally]
Reviews (1): Last reviewed commit: "fix(tables): use overflow-clip on header..." | Re-trigger Greptile |
Summary
truncate(overflow: hidden) withoverflow-clip text-ellipsis whitespace-nowrapon table header text spansoverflow: hiddencreates a scroll port that traps horizontal wheel events, preventing scrolling when hovering over header textType of Change
Testing
Tested manually
Checklist