Skip to content

Commit 8cf06ca

Browse files
committed
fix(resources): tint the list region on drop instead of ringing it
1 parent 94127c6 commit 8cf06ca

1 file changed

Lines changed: 7 additions & 6 deletions

File tree

  • apps/sim/app/workspace/[workspaceId]/components/resource

apps/sim/app/workspace/[workspaceId]/components/resource/resource.tsx

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -449,15 +449,16 @@ const ResourceTable = memo(function ResourceTable({
449449
</div>
450450
{bodyDrop?.isActive && (
451451
/**
452-
* A sibling overlay rather than an outline on the scroll container: an outline there is
453-
* painted at the scrollport edge, so the parent's `overflow-hidden` shaves its corners
454-
* and it cannot be rounded to match the rest of the surface. Inset a few pixels and
455-
* absolutely positioned, the ring keeps whole rounded corners, sits clear of the
456-
* scrollbar, and stays put while the list scrolls under it.
452+
* A soft tint over the whole list region, not a line around it. This is the workflow
453+
* sidebar's own drop-inside affordance (`bg-[var(--text-subtle)] opacity-10`), and it
454+
* is the right weight here: a hairline stretched around the entire pane reads as a
455+
* window border rather than a drop target, and being painted at the scrollport edge it
456+
* also got its corners shaved by the parent's `overflow-hidden`. A fill has no corners
457+
* to clip and no edge to fight the surrounding chrome.
457458
*/
458459
<div
459460
aria-hidden
460-
className='pointer-events-none absolute inset-1.5 rounded-[10px] border border-[var(--text-subtle)]'
461+
className='pointer-events-none absolute inset-0 bg-[var(--text-subtle)] opacity-10'
461462
/>
462463
)}
463464
{overlay}

0 commit comments

Comments
 (0)