UI for next/previous button is not proper for find in page box.
Reported by
vku...@etouch.net,
Mar 10 2016
|
|||||||||||||||||||||
Issue descriptionChrome Version: 51.0.2673.0 (Official Build)Revision 764e6bb1b05458cde3ca0702ceb801d25c3dcc3b-refs/heads/master@{#380313} (64-bit) OS:Mac Precondition: Select 'Material' option for 'Material design in the browser's top chrome. What steps will reproduce the problem? 1.Launch chrome and open NTP. 2.Press cmd+F key from keyboard to open find in page and observe on top/bottom of next/previous button. Actual: UI for next/previous button is not proper for find in page (i.e vertical line is not visible instead only dot is seen) Expected: UI for next/previous button should be properly displayed. This is a regression issue broken in 'M51' and will soon update other info.
,
Mar 10 2016
,
Mar 10 2016
Adding release block label, please undo if not the case.
,
Mar 10 2016
This is not a regression, per se - it looks like the vertical line is not being drawn when you activate the Material Design flag. This is not code I have worked on or changed so I will have to dig around for the cause or the proper owner.
,
Mar 10 2016
Actually, can you find someone to look into this? This is not code that I touched. I'm assuming that the in-page find bar is looking for the Material Design flag and drawing itself a bit differently when it's turned on. Someone who has worked on the in-page find code should take a look at it.
,
Mar 11 2016
@msw: Adding you in the loop, request you to please take a look into it. Please help us in reassign to the right owner for the issue raised.
,
Mar 14 2016
estade, pkasting?
,
Mar 14 2016
I only ever touched the Views version of the find in page bar. I don't think anyone has touched the mac FIP code recently, so it seems like shrike's work is the likeliest culprit (or at least qualifies him as a good candidate for tracking down the source of the regression, if it wasn't r378367)
,
Mar 14 2016
Only a note: The separator is also missing in Non-MaterialDesign Mode.
,
Mar 14 2016
This us not a result of any of my changes, and should be assigned to someone familiar with the Find in page code for triage.
,
Mar 14 2016
... and already broken in Chrome Stable Version 49.0.2623.87 (64-bit)
,
Mar 14 2016
,
Mar 14 2016
,
Mar 15 2016
With response to comment #11 & 13 Rechecked again and found that its seen from M30 series(i.e 30.0.1549.0) with following steps. 1.Launch chrome and open NTP. 2.Press cmd+F key and enter any letter e.g 't', observe separator 3.Enter complete letter e.g 'test' and observe separator Actual: 1.Separator appears after entering single letter i.e after step 1&2 2.It vanishes once complete text 'test/tes' is entered. Please refer attached screenshot & video
,
Mar 15 2016
Removing milestone, P3 (since it's been around for 20 milestones already).
,
Mar 17 2016
,
Mar 17 2017
This issue has been available for more than 365 days, and should be re-evaluated. Please re-triage this issue. The Hotlist-Recharge-Cold label is applied for tracking purposes, and should not be removed after re-triaging the issue. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Mar 20 2017
,
Mar 20 2017
,
Apr 16 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 16 2018
,
Nov 14
Updates: Rechecked the above issue on Mac (10.13.1, 10.13.6, 10.14.2) OS using latest Canary build #72.0.3609.3 and the issue not reproducible. New Find in bubble is seen, hence issue is not reproducible. Kindly refer the attached screen-cast. Thank You |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by vku...@etouch.net
, Mar 10 2016Status: Assigned (was: Unconfirmed)
253 KB
253 KB View Download