New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 664074 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression:Letter on tooltip is not displayed properly after mouse hover.

Reported by jshan...@etouch.net, Nov 10 2016

Issue description

Chrome 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.



 

Comment 1 by jshan...@etouch.net, Nov 10 2016

Manual regression range:
Good Build: 54.0.2836.0 
Bad Build:  54.0.2837.0
Actual_Search.png
60.6 KB View Download
Labels: Needs-Bisect
Status: Untriaged (was: Unconfirmed)
Cc: rbasuvula@chromium.org
Labels: -Needs-Bisect hasbisect-per-revision
Owner: tommycli@chromium.org
Status: Assigned (was: Untriaged)
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.
Project Member

Comment 4 by bugdroid1@chromium.org, 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

Comment 5 by dpa...@chromium.org, Nov 15 2016

Labels: Proj-MaterialDesign-WebUI
Cc: pbomm...@chromium.org gov...@chromium.org
Labels: -M-54 M-55 ReleaseBlock-Stable
M54 is already in Stable now, may be we could take the fix for M55.Lopping to folks who are involved.

Comment 7 by gov...@chromium.org, 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).
Status: Fixed (was: Assigned)
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.
Labels: Merge-TBD
[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.
Labels: -M-55 -Merge-TBD
I removed the merge and M55 labels. No need for m55 merge

Sign in to add a comment