feat: show component details on list item hover#1750
Merged
maxy-shpfy merged 1 commit intomasterfrom Feb 19, 2026
Merged
Conversation
maxy-shpfy
force-pushed
the
02-04-feat_show_component_details_on_list_item_hover
branch
2 times, most recently
from
45ef8ce to
16bb61b
Compare
maxy-shpfy
commented
Feb 4, 2026
Comment on lines
26
to
28
| const delaySetIsOpen = debounce((open: boolean) => { | ||
| setIsOpen(open); | ||
| }, HOVER_DELAY_MS); |
Collaborator
Author
There was a problem hiding this comment.
This is a smallish workaround instead of getting https://ui.shadcn.com/docs/components/base/hover-card - I decided to try a small but mighty solution before getting a bigger component.
maxy-shpfy
force-pushed
the
02-04-feat_show_component_details_on_list_item_hover
branch
from
16bb61b to
21c01b4
Compare
Mbeaulne
approved these changes
Feb 19, 2026
Comment on lines
+1
to
+13
| export const debounce = <F extends (...args: Parameters<F>) => ReturnType<F>>( | ||
| func: F, | ||
| waitFor: number, | ||
| ) => { | ||
| let timeout: ReturnType<typeof setTimeout>; | ||
|
|
||
| const debounced = (...args: Parameters<F>) => { | ||
| clearTimeout(timeout); | ||
| timeout = setTimeout(() => func(...args), waitFor); | ||
| }; | ||
|
|
||
| return debounced; | ||
| }; |
Collaborator
There was a problem hiding this comment.
[Re: lines +1 to +14]
Likely for another PR, but there are likely a few places in our code that this would work.
See this comment inline on Graphite.
Collaborator
There was a problem hiding this comment.
We have a useDebouncedState hook but it might be a bit much for this specific use case
src/components/shared/ReactFlow/FlowSidebar/components/ComponentItem.tsx
Outdated
Show resolved
Hide resolved
camielvs
approved these changes
Feb 19, 2026
Collaborator
camielvs
left a comment
camielvs
left a comment
There was a problem hiding this comment.
I'm not a fan of the UI of the popover. It feels... flat? I won't block though, we can iterate later.
You may also want to look into the failing e2e tests before merging.
maxy-shpfy
force-pushed
the
02-04-feat_show_component_details_on_list_item_hover
branch
from
21c01b4 to
617d59d
Compare
Collaborator
Author
Merge activity
|
maxy-shpfy
deleted the
02-04-feat_show_component_details_on_list_item_hover
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Added a hover popover for components in the Flow Sidebar that displays component details. When hovering over a component (i) icon in the sidebar, a popover appears showing the component name and details.
Type of Change
Checklist
Changes
ComponentHoverPopovercomponent that shows component details on hoverTextBlockcomponent to accept adefaultCollapsedpropTaskDetailsto support expanded descriptions by defaultTest Instructions
Component Details on info hover.mov (uploaded via Graphite)