Skip to content

Commit 8937d55

Browse files
committed
Dashboard: Fix table row not clickable in safari because of sticky thead (#6783)
<!-- ## title your PR with this format: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes" If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000): ## Notes for the reviewer Anything important to call out? Be sure to also clarify these in your comments. ## How to test Unit tests, playground, etc. --> <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on updating the `className` property of the `<thead>` component in two files to simplify its styling. ### Detailed summary - Updated `className` in `<thead>` component in `apps/dashboard/src/@/components/ui/table.tsx`: - Changed from a complex string with multiple classes to a simpler string: `"relative border-b bg-background"`. - Applied the same change in `apps/playground-web/src/components/ui/table.tsx`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
1 parent bf81dee commit 8937d55

File tree

2 files changed

+2
-10
lines changed

2 files changed

+2
-10
lines changed

apps/dashboard/src/@/components/ui/table.tsx

+1-5
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,7 @@ const TableHeader = React.forwardRef<
2424
>(({ className, ...props }, ref) => (
2525
<thead
2626
ref={ref}
27-
className={cn(
28-
"border-border border-b bg-background",
29-
"sticky top-0 z-20 border-none before:absolute before:inset-0 before:border-border before:border-b",
30-
className,
31-
)}
27+
className={cn("relative border-b bg-background", className)}
3228
{...props}
3329
/>
3430
));

apps/playground-web/src/components/ui/table.tsx

+1-5
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,7 @@ const TableHeader = React.forwardRef<
2424
>(({ className, ...props }, ref) => (
2525
<thead
2626
ref={ref}
27-
className={cn(
28-
"border-border border-b bg-background",
29-
"sticky top-0 z-20 border-none before:absolute before:inset-0 before:border-border before:border-b",
30-
className,
31-
)}
27+
className={cn("relative border-b bg-background", className)}
3228
{...props}
3329
/>
3430
));

0 commit comments

Comments
 (0)