Regression: Unwanted tooltip 'Settings' is displayed when mouse hover on any options from the 'Main menu' of chrome://settings/ page
Reported by
nutan.ga...@etouch.net,
Jun 27 2017
|
|||
Issue descriptionChrome Version : 61.0.3141.0 180095eb1bca7df1cdcb02547340499c2ee3af6e-refs/heads/master@{#482153} OS : Windows(7,8,10), Linux(14.04 LTS), Mac(10.12.3, 10.11.6)) What steps will reproduce the problem? 1. Launch Chrome, navigate to chrome://settings/ 2. Click on 'Main menu' and mouse hover on any options from the settings menu 3. Observe Actual: Unwanted tooltip 'Settings' is displayed when mouse hover on any options from 'Main menu' Expected: Unwanted tooltip 'Settings' should not be displayed when mouse hover on any options from 'Main menu' This is a regression issue broken in 'M61' and below is the manual regression range Good build : 61.0.3139.0 Bad build : 61.0.3140.0
,
Jun 27 2017
Adding Info.. Review-Url: https://codereview.chromium.org/2732003003
,
Jun 27 2017
Caused by my CL: "title" is overloaded in this context
,
Jun 27 2017
oh, nice... by giving the drawer a title, each <a> inherits that title as far as the UA is concerned. You'd have to explicitly add title="" to each <a> to prevent it from inheriting the title like that. https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/title ...I wonder why on earth that's useful.
,
Jun 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/35483c504bb197da94e4f4093426c7d26023e24f commit 35483c504bb197da94e4f4093426c7d26023e24f Author: hcarmona <hcarmona@chromium.org> Date: Wed Jun 28 02:14:41 2017 MD-Settings: Don't use title as a property because it's overloaded. R=michaelpg@chromium.org BUG= 737077 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/2958153002 Cr-Commit-Position: refs/heads/master@{#482853} [modify] https://crrev.com/35483c504bb197da94e4f4093426c7d26023e24f/chrome/browser/resources/md_extensions/manager.html [modify] https://crrev.com/35483c504bb197da94e4f4093426c7d26023e24f/chrome/browser/resources/md_history/app.html [modify] https://crrev.com/35483c504bb197da94e4f4093426c7d26023e24f/chrome/browser/resources/settings/settings_ui/settings_ui.html [modify] https://crrev.com/35483c504bb197da94e4f4093426c7d26023e24f/ui/webui/resources/cr_elements/cr_drawer/cr_drawer.html [modify] https://crrev.com/35483c504bb197da94e4f4093426c7d26023e24f/ui/webui/resources/cr_elements/cr_drawer/cr_drawer.js
,
Jun 30 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by hdodda@chromium.org
, Jun 27 2017Labels: hasbisect-per-revision
Owner: hcarmona@chromium.org
Status: Assigned (was: Unconfirmed)