New issue
Advanced search Search tips

Issue 618803 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 12
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

Soft undo keyboard shortcut is not working in devtools Sources panel

Reported by cont...@jaygeorge.co.uk, Jun 9 2016

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2761.0 Safari/537.36

Steps to reproduce the problem:
1. In sources view press cmd + d to highlight the next occurrence
2. Press cmd + u to try and soft undo selection

What is the expected behavior?
Devtools un-lights the last selection, as per the soft undo command.

What went wrong?
The soft undo command no longer works. Nothing happens.

Did this work before? Yes About a month or two ago

Chrome version: 53.0.2761.0  Channel: n/a
OS Version: OS X 10.11.5
Flash Version: Shockwave Flash 22.0 r0
 

Comment 1 by l...@chromium.org, Jun 9 2016

Owner: l...@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 2 by l...@chromium.org, Jun 10 2016

Labels: -Pri-2 Pri-1
Summary: Regression: Soft undo keyboard shortcut is not working in devtools Sources panel (was: Soft undo keyboard shortcut is not working in devtools Sources panel)

Comment 3 by l...@chromium.org, Jun 14 2016

Thank you for reporting.  I was able to repro earlier this week, but after trying again on Mac, Canary 53.0.2766.0, I'm unable to repro.

Are you still seeing it broken after updating?
I was on 53.0.2765.0 yesterday. I just upgraded to 53.0.2766.0 and it's
still broken for me.

Jay

Comment 5 by l...@chromium.org, Jun 21 2016

Labels: -Pri-1 Pri-2
Summary: Soft undo keyboard shortcut is not working in devtools Sources panel (was: Regression: Soft undo keyboard shortcut is not working in devtools Sources panel)
I've tried out soft undo and found it working (not broken) in several cases.  In the cases I've tried, it seems that this bug only consistently appears on pretty-fied formatted files and HTML files.  For example repro, soft undo is broken on "http://asdf.com/" in the "(index)" source and on "ga.js" in the google-analytics folder after you click the Pretty Print button.

I can get started on these 2 cases.  It looks like they've been broken before, so this is actually not a regression.

Could you please let me know if there are other non-HTML or non-prettified cases where soft undo is broken?  Thanks!
I just tried on 53.0.2774.2 and it's still broken for me.
I've attached a video to show it broken.
I've also just tried in Chrome stable (51.0.2704) and it is broken there too.

Are you a different OS per chance?
Soft Undo Keyboard Shortcut not working.mov
17.0 MB Download

Comment 7 by l...@chromium.org, Jun 21 2016

Status: Started (was: Assigned)
Thank you for the helpful screencast!  I can confirm this bug on Stable and Canary as well, Linux and Mac.  It seems that only readonly files are affected.

CL in review
https://codereview.chromium.org/2080603008/

Comment 8 by l...@chromium.org, Jun 27 2016

Comment 9 by l...@chromium.org, Jun 27 2016

Status: ExternalDependency (was: Started)

Comment 10 by l...@chromium.org, Jun 30 2016

Status: Assigned (was: ExternalDependency)
CodeMirror v5.16.2 has fixed soft-undo for read-only files.  Now we just need to bump up the version in DevTools.
Owner: einbinder@chromium.org
FYI I'm not sure what you mean by "read only files" are affected.
I'm working on live projects in the "Sources" panel and writing to disk, and this problem is still there in Chrome Canary 54.0.2834.0
This still broken, over a year later, even though it's an advertised keyboard shortcut.

Reading through the thread, I think you misdiagnosed — This is not to do with "readonly files", it is affecting the main "text editor" in DevTools.

To show that it's broken, I've attached a video of a very simple 10 second demo in the latest DevTools.
Soft Undo.mp4
1.4 MB View Download
Status: Fixed (was: Assigned)
Could you let me know which version you've fixed this in so I can verify the fix? Thanks

Sign in to add a comment