-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Refactores the Tooltip component into a functional component #27353
Conversation
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.
LGTM, tested and works as expected. @grzim check if the tests pas and maybe fix that nitpick I left about that old comment.
} | ||
function Tooltip( { children, position, text, shortcut } ) { | ||
/** | ||
* Whether a the mouse is currently pressed, used in determining whether |
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.
"Whether a the mouse" => "Whether the mouse"
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.
comment fixed,
tooltip unit tests are passing
04b5d36
to
be513bc
Compare
resolved in #27682 |
Description
Regarding #22890 "Refactor all React class components to functional components using hooks" this pull request contains a rewritten tooltip
How has this been tested?
Changes tested visually and with a set of unit tests that were adjusted to work with the functional component.
Screenshots
Types of changes
Refactoring
Checklist: