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

Issue 783090 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

In 'Filter bar'/TextPrompt bar, caret gets misplaced after pressing down arrow key.

Reported by avsha...@etouch.net, Nov 9 2017

Issue description

Chrome version : 64.0.3263.0 (Official Build) fd1e89557fa78a467061f732a8473fd863a89b1a-refs/heads/master@{#515052} 32/64-bit
OS : Windows(7,8,10), Mac(10.12.6), Linux(14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome and open devtools on NTP.
2. Navigate to 'Console' tab and enter 'S' in 'Filter' text field.
3. Press down arrow key from keyboard and observe the text caret in  'Filter' text field.

Actual Result : Text caret gets misplaced after pressing down arrow key.

Expected Result : Text caret in 'Filter' bar should not get misplaced on pressing down arrow key.

This is a regression issue broken in ‘M-64’ and using the per-revision bisect providing the bisect results,
Good build : 64.0.3240.0 (Revision : 508893)
Bad build : 64.0.3241.0 (Revision : 508934)

You are probably looking for a change made after 508916 (known good), but no later than 508917 (first known bad).

CHANGELOG URL:
https://chromium.googlesource.com/chromium/src/+log/0ff7688d1d4cb6da35e5989462d1d54488ddabf1..1eff80a625bd2a48a24bd2927ac67468489e8e61

Suspect : https://chromium.googlesource.com/chromium/src/+/1eff80a625bd2a48a24bd2927ac67468489e8e61

@luoe : Could you please look into the issue, pardon me if it has nothing to do with your changes and if possible please assign it to concern owner.
 
Actual_Text_Caret.mp4
875 KB View Download
Expected_Text_Caret.mp4
679 KB View Download

Comment 1 by l...@chromium.org, Nov 10 2017

Cc: l...@chromium.org
Labels: -Pri-1 Pri-2
Owner: einbinder@chromium.org
Summary: In 'Filter bar'/TextPrompt bar, caret gets misplaced after pressing down arrow key. (was: Regression : In 'Filter' bar, text caret gets misplaced after pressing down arrow key.)
Thanks for the report.

I'm able to repro in the console filter bar, and also in other TextPrompts when I press ArrowDown more than once (in Network's filter bar, ClassesPaneWidget).

einbinder@, could you please take a look?  I'm guessing the caret is able to land to the right of the ghostTextElement somehow.  Maybe we could move the ghostTextElement outside of the contentEditable div?
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 10 2017

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

commit 3c03232fec035fc3164caa768b5f77e1a56bad33
Author: Joel Einbinder <einbinder@chromium.org>
Date: Fri Nov 10 23:24:12 2017

DevTools: Accept and clear autocomplete with ArrowUp/ArrowDown

ArrowDown moves the cursor the right, so it needs to accept the
autocomplete otherwise the cursor will be to the right of the ghost
text.

Bug:  783090 
Change-Id: I862fb2ae37d79dcd5f507087717d16899220419a
Reviewed-on: https://chromium-review.googlesource.com/762880
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Reviewed-by: Erik Luo <luoe@chromium.org>
Commit-Queue: Joel Einbinder <einbinder@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515740}
[modify] https://crrev.com/3c03232fec035fc3164caa768b5f77e1a56bad33/third_party/WebKit/Source/devtools/front_end/ui/TextPrompt.js

Comment 3 by avsha...@etouch.net, Nov 13 2017

Labels: TE-Verified-64.0.3267.0 TE-Verified-M64
Update : 
Retested above issue on Windows(7,8,10), Mac(10.12.6) & Linux(14.04 LTS) OS using latest Canary #64.0.3267.0 and issue is fixed. Kindly review an attached screen cast.

Thank you!
Canary_behaviour.mp4
959 KB View Download

Comment 4 by avsha...@etouch.net, Nov 13 2017

Note : 
Mac build #64.0.3267.0 is failed today, hence retested above issue using latest Mac canary #64.0.3265.0 build and the issue is no longer reproducible. The fix is working as intended. 

Thank you!
Mac_behaviour.mov
3.8 MB Download

Comment 5 by l...@chromium.org, Nov 13 2017

Status: Fixed (was: Assigned)
Thanks for verifying, and thanks einbinder@ for the fix!

Sign in to add a comment