Regression:Letter on tooltip is not displayed properly after mouse hover.
Reported by
jshan...@etouch.net,
Nov 10 2016
|
||||||||
Issue descriptionChrome Version: 56.0.2914.3 (Official Build) b57184fe310b2dd87729c8b08e3e30dde2394195-refs/branch-heads/2914@{#7} 32/64-bit OS: Mac(10.11.6, 10.12.1), Windows (7,8,8.1,10), Linux (14.04 LTS) What steps will reproduce the problem? (1)Launch chrome and navigate to chrome://md-settings/ (2)Click on 'search settings' field and enter alphabet'y' or 'j' (3)Now hover mouse on tooltip such that it shift up and observe the letter on tooltip. Actual: Letter on tooltip is not displayed properly after mouse hover. Expected: Letter on tooltip should be completely visible after mouse hover. This is a regression issue broken in 'M54' and will soon update other info.
,
Nov 10 2016
,
Nov 11 2016
Using the per-revision bisect providing the bisect results, Good build: 54.0.2836.0 (Revision: 413363). Bad build: 54.0.2837.0 (Revision: 413618). You are probably looking for a change made after 413581 (known good), but no later than 413582 (first known bad). CHANGE-LOG URL: --------------------------------------- https://chromium.googlesource.com/chromium/src/+log/2a8aef55976f4a22a32f1ee79095f2c5a7c3ee8a..1886c57203a481f6818529a50c82dcc753dacdef From the CL above, assigning the issue to the concern owner @ tommycli : ------------------ Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner. Review-Url: https://codereview.chromium.org/2260373002 Note : Able to reproduce the issue in Ubuntu 14.04, Win 10.0 & Mac 10.11.6.
,
Nov 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8cc356c754f7f9632c538b0f7318e42d97573ad0 commit 8cc356c754f7f9632c538b0f7318e42d97573ad0 Author: tommycli <tommycli@chromium.org> Date: Tue Nov 15 01:53:20 2016 MD Settings: Fix Search tooltip arrow obscuring letters The arrow for the search bubbles is rendering on top of the text. This blocks parts of the letter for letters like 'y' or 'j', when the tooltip is in the "above" position. This CL adds a z-index to put the arrow behind the letters, which is the same solution that Old Options used. BUG= 664074 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2495373004 Cr-Commit-Position: refs/heads/master@{#432054} [modify] https://crrev.com/8cc356c754f7f9632c538b0f7318e42d97573ad0/chrome/browser/resources/settings/settings_shared_css.html
,
Nov 15 2016
,
Nov 15 2016
M54 is already in Stable now, may be we could take the fix for M55.Lopping to folks who are involved.
,
Nov 15 2016
A friendly reminder that M55 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you! Also due to Thanksgiving holidays in US, please make sure fix is ready and merged to M55 latest by 5:00 PM PT Friday, 11/18/16 (sooner the better).
,
Nov 15 2016
Hey guys, this does not need a merge, since MD Settings isn't released yet. The fix will go out as normal along with M56.
,
Nov 15 2016
[Auto-generated comment by a script] We noticed that this issue is targeted for M-55; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-55 label, otherwise remove Merge-TBD label. Thanks.
,
Nov 15 2016
I removed the merge and M55 labels. No need for m55 merge |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by jshan...@etouch.net
, Nov 10 201660.6 KB
60.6 KB View Download