Bug 137467

Summary:

Web Inspector: Cursor and scroll view jumps around when modifying styles

Product:

WebKit

Reporter:

Jerrold Poh <jerrold.poh>

Component:

Web Inspector

Assignee:

Timothy Hatcher <timothy>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bugzilla, commit-queue, graouts, joepeck, timothy, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Mac (Intel)

OS:

Other

Attachments:

Description	Flags
Shows position of dock and window of when bug can be replicated	none
Patch	none
Patch	none

Jerrold Poh

Reported 2014-10-06 15:37:56 PDT

Created attachment 239362 [details] Shows position of dock and window of when bug can be replicated Seen on OS X Yosemite GM (14A379a), Safari 8 (10600.1.25). To replicate, load the web inspector for a page and inspect an element. Ensure the web inspector is docked. - When enabling / disabling a style, scroll area jumps around - When adding or modifying a style, as you start typing sometimes characters are "skipped". E.g. typing "margin-top" will sometimes result in "marin-top" as characters are dropped - Cursor also sometimes jumps to start / end of line sporadically when editing - And scroll view of dock will scroll up and down too Could be related to the autocomplete feature, as I notice if I type slowly and use the auto complete, I can sometimes prevent the cursor jumps / skips See attachment which shows position of dock and window of when bug can be replicated.

Attachments
Shows position of dock and window of when bug can be replicated (574.19 KB, image/png) 2014-10-06 15:37 PDT, Jerrold Poh	no flags	Details
Patch (3.88 KB, patch) 2014-11-06 14:59 PST, Timothy Hatcher	no flags	Details Formatted Diff Diff
Patch (6.20 KB, patch) 2014-11-07 10:31 PST, Timothy Hatcher	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2014-10-06 15:38:35 PDT

<rdar://problem/18561845>

Timothy Hatcher

Comment 2 2014-11-06 14:56:32 PST

This turns out to be a CodeMirror bug. I have a fix. I also filed a bug for CodeMirror: https://github.com/codemirror/CodeMirror/issues/2912

Timothy Hatcher

Comment 3 2014-11-06 14:59:39 PST

Created attachment 241133 [details] Patch

Timothy Hatcher

Comment 4 2014-11-07 10:31:45 PST

Created attachment 241190 [details] Patch

Joseph Pecoraro

Comment 5 2014-11-07 13:24:07 PST

Comment on attachment 241190 [details] Patch r=me. Never thought I'd review a patch that used the word "blinky" in it.

WebKit Commit Bot

Comment 6 2014-11-07 14:01:53 PST

Comment on attachment 241190 [details] Patch Clearing flags on attachment: 241190 Committed r175763: <http://trac.webkit.org/changeset/175763>

WebKit Commit Bot

Comment 7 2014-11-07 14:01:56 PST

All reviewed patches have been landed. Closing bug.

Michael Bailey

Comment 8 2015-01-07 12:00:08 PST

I’m still seeing this in both Safari 8.0.2 (10600.2.5) and WebKit r178028. Screen recording of the issue, taken with WebKit r178028: http://roguewhaler.net/files/web-inspector-jumping.mov Should I file a new bug for this?

Timothy Hatcher

Comment 9 2015-01-07 12:24:17 PST

Yes, please do.

Michael Bailey

Comment 10 2015-01-07 12:40:47 PST

Submitted: https://bugs.webkit.org/show_bug.cgi?id=140193

Note You need to log in before you can comment on or make changes to this bug.