fix: fix scrolling header visual broken layout in templates#449
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
LGTM — minor Tailwind class tweak on the header row to fix a visual layout issue, mirrored in both tables.
Extended reasoning...
Overview
This PR adds Tailwind utility classes (-mx-2 px-2 w-[calc(100%+16px)]) to the DataTableRow inside the sticky table header for two templates tables (templates/builds/table.tsx and templates/list/table.tsx). The change is purely a CSS/styling fix for a visual layout issue when the header scrolls, with no logic changes.
Security risks
None — purely a CSS class change with no impact on auth, data flow, or user input handling.
Level of scrutiny
Low. This is a visual-only fix scoped to two header rows, with before/after screenshots in the PR description that demonstrate the intent. The change is symmetrically applied to both tables and matches an obvious pattern (compensating for the parent's horizontal padding so the sticky header spans the full width).
Other factors
No bugs were flagged by the bug hunting system, the Vercel preview deploy is Ready, and a Linear ticket (EN-1069) is linked. Self-contained and mechanical — appropriate to shadow-approve.
Kraci
deleted the
fix-scrolling-header-visual-broken-layout-in-templates-en-1069
branch
2 participants
Context: Quick fix to remove customer-facing visual bug
Follow-up: Create a unified table hover experience in the design system for both table headers and rows