We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Add tooltips in a architectural clever way.
Ignore shortcut texts for now, they'll come later (among other things depending on ampproject/amp-wp#3826).
Architecture could be a new top-level tooltip service and a useTooltip hook exposing a showTooltip function that accepts a DOM element and a string.
useTooltip
showTooltip
Alternatively it could be an element, that buttons and icons can be wrapped in such as:
<Tooltip text="Duplicate page"> <Button icon="trash" /> </Tooltip>
Do not alter or remove anything below. The following sections will be managed by moderators only.
The text was updated successfully, but these errors were encountered:
wassgha
Successfully merging a pull request may close this issue.
Feature description
Add tooltips in a architectural clever way.
Ignore shortcut texts for now, they'll come later (among other things depending on ampproject/amp-wp#3826).
Architecture could be a new top-level tooltip service and a
useTooltip
hook exposing ashowTooltip
function that accepts a DOM element and a string.Alternatively it could be an element, that buttons and icons can be wrapped in such as:
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation brief
QA testing instructions
Demo
Changelog entry
The text was updated successfully, but these errors were encountered: