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

Issue 607431 link

Starred by 2 users
Status: Verified
Owner:
l...@chromium.org
Closed: May 2016
Cc:
nyerramilli@chromium.org
█ tkonch...@chromium.org
l...@chromium.org
█ ashej...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Cursor in search field is not visible after resizing dev tools window.

Reported by rk...@etouch.net, Apr 28 2016 
Chrome Version: 52.0.2719.0 Revision db7757b63f29696b9b465949f9704a0d9d149f2b-refs/heads/master@{#390251}
OS: Windows (7,8,8.1,10), Mac

What steps will reproduce the problem?
(1) Launch chrome, open dev tools window and press Esc key.
(2) Open Search section, resize dev tools window to right side and click on search field.
(3) Observe the cursor in search field.

Cursor in search field is not visible after resizing dev tools window.

Cursor in search field should be visible after resizing dev tools window.

This is  a regression issue, broken in 'M-52', below is bisect info:

Good Build: 52.0.2718.0
Bad Build: 52.0.2719.0

Narrow Bisect :
https://chromium.googlesource.com/chromium/src/+log/66bea1de4dbc4a18684655979360031c14ad97a9..f1afe5ba38f00c14f9982c8d0b3afd598865f411?pretty=fuller&n=100

Suspecting: r390177
Actual_Cursor.mp4
1001 KB Download
Expected_Cursor.mp4
551 KB Download

Comment 1 by tkonch...@chromium.org, Apr 28 2016

Labels: ReleaseBlock-Stable
Adding RB label as this is a recent regression.

Comment 2 by l...@chromium.org, Apr 29 2016

Cc: chowse@chromium.org maxwalker@chromium.org
@maxwalker, @chowse
For this detail, what do you think about using min-width: 95px to make sure that a portion of the search bar text is always visible?
cursorminwidth.png
10.0 KB View Download
Project Member

Comment 3 by bugdroid1@chromium.org, May 2 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/320c6f9e6c7d6104a60d7505957bfe2634a42e1f

commit 320c6f9e6c7d6104a60d7505957bfe2634a42e1f
Author: luoe <luoe@chromium.org>
Date: Mon May 02 19:52:12 2016

DevTools: fix visual regressions in full text search

BUG= 607431 , 607432 

Review-Url: https://codereview.chromium.org/1925253003
Cr-Commit-Position: refs/heads/master@{#391033}

[modify] https://crrev.com/320c6f9e6c7d6104a60d7505957bfe2634a42e1f/third_party/WebKit/Source/devtools/front_end/sources/sourcesSearch.css

Comment 4 by l...@chromium.org, May 2 2016

Status: Fixed (was: Assigned)
Changes committed.  Please review if needed.

Comment 5 by l...@chromium.org, May 17 2016

Cc: -maxwalker@chromium.org -chowse@chromium.org
Status: Started (was: Fixed)
Issue re-appeared since revert of https://codereview.chromium.org/1950343003

Comment 6 by l...@chromium.org, May 17 2016

Cc: ashej...@chromium.org l...@chromium.org
 Issue 609013  has been merged into this issue.

Comment 7 by brajkumar@chromium.org, May 18 2016

Labels: TE-Verified-M52 TE-Verified-52.0.2740.0
Tested this issue on Windows 7, Ubuntu 14.04 and Mac OS 10.11.4 using chrome latest canary M52-52.0.2740.0 by followings steps mentioned in the comment #0. Observed that the cursor in search field is now visible after resizing dev tools window as expected. Hence adding TE-Verified label.

Thanks!
Cursor.mp4
1.1 MB Download

Comment 8 by l...@chromium.org, May 19 2016

Status: Verified (was: Started)
Thanks for the verification!  Closing ticket
►

Sign in to add a comment