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

Issue 607421 link

Starred by 1 user
Status: Fixed
Owner:
l...@chromium.org
Closed: Apr 2016
Cc:
nyerramilli@chromium.org
█ tkonch...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Faint text in search field is not seen properly after changing writing direction in dev tools.

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, right click in text field and change writing direction Right to Left.
(3) observe the faint text in field

Incorrect position of round brackets is seen faint text of search field after changing writing direction.

Position of round brackets of faint text in search field should be proper after changing writing direction.

This is  a regression issue, broken in 'M-52', will soon update the other info:

Comment 1 by rk...@etouch.net, Apr 28 2016

Labels: hasbisect OS-Linux
Owner: l...@chromium.org
Status: Assigned (was: Unconfirmed)
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_FaintText.mp4
589 KB Download
Expected_FaintText.mp4
467 KB Download

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

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

Comment 3 by l...@chromium.org, Apr 28 2016

Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 29 2016 

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

commit b7ebd042d13e4fdc1d6466846baf5cc431b2a557
Author: luoe <luoe@chromium.org>
Date: Fri Apr 29 17:57:53 2016

DevTools: add LRM character to correct search placeholder text

For more info on this particular RTL behavior, see:
http://stackoverflow.com/questions/5801820/how-to-solve-bidi-bracket-issues

BUG= 607421 

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

[modify] https://crrev.com/b7ebd042d13e4fdc1d6466846baf5cc431b2a557/third_party/WebKit/Source/devtools/front_end/sources/AdvancedSearchView.js

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

Status: Fixed (was: Started)
This should be addressed on tip of tree.  Please verify if needed.
►

Sign in to add a comment