Material settings search: the tooltip for a nested occurrence obscures next item
Reported by
woxxom@gmail.com,
Jun 13 2017
|
||||||||
Issue description
UserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3129.0 Safari/537.36
Steps to reproduce the problem:
1. open browser settings
2. type "tab" without quotes in the search input box
What is the expected behavior?
The item after the one with a tooltip ("Manage other people") is fully readable
What went wrong?
"Manage other people" is almost completely obscured by the yellow tooltip. To see what was found we have to point the mouse cursor over the item so that the tooltip flips to the other side.
Did this work before? No
Chrome version: 61.0.3129.0 Channel: canary
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version:
The tooltip can be moved further to the right so that we can read the most important part of the label - its start. The tooltip can be moved higher since currently it misleadingly appears inside the next item.
,
Jun 14 2017
,
Jun 14 2017
Issue is reproducible on the latest canary(61.0.3130.0) on Windows-10, Mac OS 10.12.5 and Linux Ubuntu 14.04.
,
Jun 14 2017
Note: Issue is non-regression as similar behavior is observed on older chrome version: 55.0.2883.87(search setting feature on material designed page was no supported on 53.0.2784.0)
,
Jun 14 2017
The tooltip moves away once you hover it with the mouse, to address exactly that problem (which is same as old settings).
,
Jun 14 2017
#5, I know it moves away on hover. I understand your reluctance to improve on this especially if you were the one who implemented it, but let's be reasonable: * the tooltip in the old UI was semi-transparent so the underlying text was visible, even if not exactly as easily as unobscured. * currently the tooltip is flipped when it's hovered, not when the obscured item's text is hovered. It looks weird as the tooltip escapes from the mouse cursor like in "press-to-continue-trial-period" nags that some commercial applications of the 90's were showing. This is not how things are done in modern UI, especially in a desktop app. * I see no valid reason not to move the tooltip e.g. reduce the vertical padding by a few pixels and raise it by a few pixels. See the attachment.
,
Jun 14 2017
> I understand your reluctance to improve on this especially if you were the one who implemented it This logic seems exactly reverse from what I would expect. If I am the one who implemented it, and agree with the suggested improvements, I would be the first one trying to implement them. I don't think such comments facilitate the discussion, so let's keep the discussion about the UI itself. @bettes: Alan can you take a look at these suggestions?
,
Jun 14 2017
FWIW, I meant I understand the reported issue is rather unimportant so dismissing it based on the report alone isn't surprising, especially since the original post didn't elaborate on the reasons and proposed changes.
,
Jun 19 2017
From our UI sync: - Change tooltip to 0.9 opacity - Change from fixed width (100px) to min-width: 64px and max-width: 100px
,
Jul 6 2017
,
Jul 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b36f4aaf404427cef20af39e7b657f9afe942ae0 commit b36f4aaf404427cef20af39e7b657f9afe942ae0 Author: dpapad <dpapad@chromium.org> Date: Fri Jul 07 20:41:20 2017 MD Settings: Tweak search bubble opacity and size. BUG= 732795 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2974583002 Cr-Commit-Position: refs/heads/master@{#485034} [modify] https://crrev.com/b36f4aaf404427cef20af39e7b657f9afe942ae0/chrome/browser/resources/settings/settings_shared_css.html
,
Jul 7 2017
Marking this as fixed. Changes mentioned at comment #9 have landed. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by woxxom@gmail.com
, Jun 13 2017