-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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] Fix RTL-toggle tooltip bug in app bar #17865
Conversation
Details of bundle changes.Comparing: 83b8323...58827e1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for looking at the problem! I think that we can actually close the issue after handling the documentation issue :). Issue's codesandbox was fixed during the class to hook migration @joshwooding did.
I have proposed a one-line fix, let me know what you think about this solution (the last commit).
@oliviertassinari if you can do it in one line that's fantastic! |
@flurmbo Nice, thanks. |
This fixes an issue related to #13203 in the documentation site's appbar. Currently when you click the button to toggle right-to-left, the tooltip message persists after clicking. This change manually controls the state of the tooltip and closes it when clicked.
Closes #13203