-
Notifications
You must be signed in to change notification settings - Fork 3
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
2200 - Adding tooltip to Total messages header for explanation #2273
base: main
Are you sure you want to change the base?
Conversation
…o 2200-contextual-tooltips
line-height: units(1); | ||
} | ||
} | ||
// .usa-tooltip { |
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.
Please remove the comment if we don't need it anymore
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.
hi! i saw that my review is requested for this, and i'm wondering where i might find a screen capture of it. thank you! tho i do appreciate the clarity of .usa-tooltip being added
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!
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.
is a screen capture possible so i may see these changes? thank you much @jonathanbobel
Description
Users are confused as to what Total messages are/is in the Dashboarding. Adding a tooltip to help explain.
TODO (optional)