-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Remove Dashboard Settings
tool tip for each stats
#6084
Comments
Still reproducible
|
I'm going to get started on this. Just to be sure we want this totally removed? Or should the tooltip say something different? @rebron |
@GeetaSarvadnya if this is still relevant, I'm happy to create a PR for this one. |
@GeetaSarvadnya I think the desired behavior should be as follows:
I'm going to open a PR for these. If we don't want both of them, happy to revert the 2nd half. |
Hey @GeetaSarvadnya @btlechowski , I was seeing that the issue had a PR but it was closed, and I would like to know if it is still relevant and I can take it? My only question is, only the tooltip should be removed only from the options, in the rest of the modal it should remain right? |
please assign this issue to me |
Description
Remove
Dashboard Settings
tool tip for each statsSteps to Reproduce
Dashboard Settings
tooltip is shown for each statsActual result:
Mouseover on each stats shows
Dashboard Settings
tooltipExpected result:
Remove
Dashboard Settings
tooltip for each statsReproduces how often:
Always
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
@brave/legacy_qa @karenkliu
The text was updated successfully, but these errors were encountered: