Implement tooltip on custom call type#182
Conversation
Signed-off-by: kumaradityaraj <sedulous.0007@gmail.com>
✅ Deploy Preview for swf-editor ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR replaces the browser default title tooltip for unknown task badges with the app’s Tooltip component to provide a consistent UI.
Changes:
- Adds Tooltip UI components to
Nodes.tsx - Wraps custom/unknown badges with
TooltipTriggerandTooltipContent - Introduces inline color styling based on
prefers-color-scheme
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@lornakelly Please review this. |
| </span> | ||
| <Tooltip> | ||
| <TooltipTrigger asChild> | ||
| <span className="dec-task-node-badge-custom" data-testid={`${testId}-custom`}> |
There was a problem hiding this comment.
The span here should be removed, that was temporary for the browser tooltip , please also remove the associated css
| <TooltipContent> | ||
| <p | ||
| style={{ | ||
| color: window.matchMedia("(prefers-color-scheme: dark)").matches ? "#fff" : "#000", |
There was a problem hiding this comment.
This should be done in css and follow the same pattern the other components do for styling light and dark modes (in css files you will see classes with .dark)
Closes
A small, reusable tooltip component and use it to replace the native browser title tooltip currently shown on custom call type badges.
Note - Tooltip hover style is breaking currently. A ss has been attached.