Issue metadata
Sign in to add a comment
|
Regression: In bookmarks bar, unnecessary tooltip is seen for folders with small name.
Reported by
vvishwak...@etouch.net,
Aug 9 2016
|
||||||||||||||||||||||
Issue descriptionVersion: 54.0.2824.0 (Official Build) facabd3224aecbcab4bea9daadad31c67488d78c-refs/heads/master@{#410520} (64-bit) OS: Mac (10.10.5, 10.11.5) What steps will reproduce the problem? 1) Launch chrome, go to NTP and add a ‘New folder’ on Bookmarks bar. 2) Create another folder inside the already created folder. 3) Now click on ‘New folder’ in bookmarks bar and mouse hover on the folder inside it. When mouse hovered on ‘New folder’, unnecessary tooltip is seen. Tooltip should be seen only for folders with big names. This is a Regression issue broken in M-49, below is the bisect info Manual bisect: Good build: 49.0.2573.0 Bad build: 49.0.2574.0 Narrow bisect URL: https://chromium.googlesource.com/chromium/src/+log/6ccf5fd5ca8ee56e0de7f1b768a7e904662edc85..68cf0b8360a8ea1dbcd5efb524e7831e1c22bd66?pretty=fuller&n=10000 Suspecting: r361388 Note: Issue is not seen on Windows and Linux OS.
,
Nov 15 2016
This bug is tagged as regression.Which means that the bisects are incorrect or do not have an owner who is actively investigating. Requesting the reporter to triage and update the behavior in all the latest chrome channels and bisect if needed.Close as WontFix if not reproducible.
,
Nov 16 2016
I will handle this issue.
,
Nov 16 2016
I upload CL for this issue. (https://codereview.chromium.org/2504753003/) I also upload screen shot to show how it works.
,
Nov 21 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/efa1d76a70dfec345ff89d2ca978a46029c51e7e commit efa1d76a70dfec345ff89d2ca978a46029c51e7e Author: corona10 <corona10@gmail.com> Date: Mon Nov 21 16:38:55 2016 Bookmarks bar, unnecessary tooltip is seen for folders with small name. BUG= 635836 Review-Url: https://codereview.chromium.org/2504753003 Cr-Commit-Position: refs/heads/master@{#433563} [modify] https://crrev.com/efa1d76a70dfec345ff89d2ca978a46029c51e7e/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller.mm [modify] https://crrev.com/efa1d76a70dfec345ff89d2ca978a46029c51e7e/chrome/browser/ui/cocoa/bookmarks/bookmark_bar_folder_controller_unittest.mm
,
Nov 21 2016
Thanks for the patch! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by ellyjo...@chromium.org
, Aug 10 2016Owner: ----
Status: Available (was: Assigned)