Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Component | Tooltip: Add show/hide delay and improve hover behavior #537

Merged
merged 4 commits into from
Mar 3, 2025

Conversation

rokotyan
Copy link
Contributor

  • Add showDelay and hideDelay config options to control tooltip timing
  • ⚠️ Change default allowHover to false for better default behavior (we've introduced this feature recently so I find this small breaking change acceptable, though it's not a good practice)
  • Fix hover behavior to work independently from followCursor setting
  • Add internal timeout management for delayed show/hide
  • Hide the tooltip on component initialization
  • Clean up timeouts on tooltip destruction
Screen.Recording.2025-02-26.at.8.27.21.AM.mp4
Screen.Recording.2025-02-26.at.8.30.55.AM.mp4

- Add `showDelay` and `hideDelay` config options to control tooltip timing
- ⚠️ Change default `allowHover` to `false` for better default behavior
- Fix hover behavior to work independently from `followCursor` setting
- Add internal timeout management for delayed show/hide
- Hide the tooltip on component initialization
- Clean up timeouts on tooltip destruction
• Add new example showcasing tooltip show/hide delay functionality with configurable timing
• Update tooltip stick position example to explicitly set `allowHover` prop
…amples

• Update the hoverable content section to clarify `allowHover` usage
• Add new section documenting `showDelay` and `hideDelay` options
• Update example code to use explicit `allowHover` prop
• Add interactive example demonstrating tooltip delay functionality
• Add `showDelay` and `hideDelay` input properties to Angular tooltip component
• Update tooltip property documentation to match core component defaults
• Fix `interpolateMissingData` type in Line component to be optional
@lee00678 lee00678 merged commit a81b294 into f5:main Mar 3, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants