Open GeetaSarvadnya opened 5 years ago
Still reproducible
Brave | 1.13.82 Chromium: 85.0.4183.83 (Official Build) (64-bit) |
---|---|
Revision | 94abc2237ae0c9a4cb5f035431c8adfb94324633-refs/branch-heads/4183@{#1658} |
OS | Ubuntu 18.04 LTS |
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:
X
buttonI'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