Skip to content

Commit 96e4800

Browse files
author
Theodore Li
committed
Fix lint, attach scroll listener to tabs themselves
1 parent ba84316 commit 96e4800

File tree

1 file changed

+6
-8
lines changed
  • apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-tabs

1 file changed

+6
-8
lines changed

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-tabs/resource-tabs.tsx

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -99,20 +99,18 @@ export function ResourceTabs({
9999
const PreviewModeIcon = PREVIEW_MODE_ICONS[previewMode ?? 'split']
100100
const nameLookup = useResourceNameLookup(workspaceId)
101101
const scrollNodeRef = useRef<HTMLDivElement>(null)
102-
const wrapperRef = useRef<HTMLDivElement>(null)
103102

104103
useEffect(() => {
105-
const wrapper = wrapperRef.current
106-
const scrollNode = scrollNodeRef.current
107-
if (!wrapper || !scrollNode) return
104+
const node = scrollNodeRef.current
105+
if (!node) return
108106
const handler = (e: WheelEvent) => {
109107
if (e.deltaY !== 0) {
110-
scrollNode.scrollLeft += e.deltaY
108+
node.scrollLeft += e.deltaY
111109
e.preventDefault()
112110
}
113111
}
114-
wrapper.addEventListener('wheel', handler, { passive: false })
115-
return () => wrapper.removeEventListener('wheel', handler)
112+
node.addEventListener('wheel', handler, { passive: false })
113+
return () => node.removeEventListener('wheel', handler)
116114
}, [])
117115

118116
const addResource = useAddChatResource(chatId)
@@ -281,7 +279,7 @@ export function ResourceTabs({
281279
<p>Collapse</p>
282280
</Tooltip.Content>
283281
</Tooltip.Root>
284-
<div ref={wrapperRef} className={cn('flex min-w-0 flex-1 items-center', RESOURCE_TAB_GAP_CLASS)}>
282+
<div className={cn('flex min-w-0 flex-1 items-center', RESOURCE_TAB_GAP_CLASS)}>
285283
<div
286284
ref={scrollNodeRef}
287285
className={cn(

0 commit comments

Comments
 (0)