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

docs(react-tooltip): add Tooltip and overflow content example #33278

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mainframev
Copy link
Contributor

@mainframev mainframev commented Nov 15, 2024

Adding an additional example to our react-tooltip component, demonstrating a controlled tooltip with overflow logic. This feature was available in Fluent UI v8 through a built-in overflowMode prop, but it is not supported in the current v9 API, users should use onVisibilityChange. Partners have been asking how to replicate this functionality in v9. While the migration documentation notes the absence of overflowMode, this example aims to address the scenario.

We can approach this in two ways: either clarify that managing tooltip with overflow is now a user responsibility (as a "by design" decision) or provide this example to guide users on implementing it.

Screenshot 2024-11-15 at 18 03 56

UPD: found that onlyIfTruncated example existed before in preview version and was removed later here. We may get it back, if this will help partners with migration, let's decide.

Previous Behavior

New Behavior

Related Issue(s)

Copy link

github-actions bot commented Nov 15, 2024

📊 Bundle size report

✅ No changes found

Copy link

Pull request demo site: URL

@mainframev mainframev marked this pull request as ready for review November 15, 2024 17:14
@mainframev mainframev requested a review from a team as a code owner November 15, 2024 17:14
@mainframev mainframev force-pushed the docs/tooltip-and-overflow-example branch from d5e1d6e to 1bbb199 Compare November 15, 2024 18:21
@mainframev mainframev force-pushed the docs/tooltip-and-overflow-example branch from 1bbb199 to 48b5cd8 Compare November 15, 2024 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature]: Addition to Tooltip for managing overflow
1 participant