Bug 21000

Summary: Hover to select improvements
Product: WebKit Reporter: Matt Lilek <dev+webkit>
Component: Web Inspector (Deprecated)Assignee: Timothy Hatcher <timothy>
Status: RESOLVED FIXED    
Severity: Enhancement CC: aroben, mitz, ojan, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Proposed patch mitz: review+

Matt Lilek
Reported 2008-09-22 10:44:30 PDT
In hover to select mode, auto scrolling should be disabled so elements aren't jumping around the page as they're trying to be selected and the inspected page should be focused when select mode is started to match Firebug.
Attachments
Proposed patch (16.47 KB, patch)
2008-09-28 16:17 PDT, Timothy Hatcher 		mitz: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Timothy Hatcher
Comment 1 2008-09-22 10:47:26 PDT
Lets just disable scrolling for all highlights. We have gotten requests to remove it since it is annoying when the page moves when you are hovering the DOM tree. Maybe we want a modifier key?
mitz
Comment 2 2008-09-24 13:17:08 PDT
It would be nice, if you disabled scrolling (which I think you should) if, when the highlighted node was off-screen, an arrow pointing in its general direction would show up.
Timothy Hatcher
Comment 3 2008-09-28 16:17:24 PDT
Created attachment 23898 [details] Proposed patch
mitz
Comment 4 2008-09-28 16:26:46 PDT
Comment on attachment 23898 [details] Proposed patch + invasive and causes less thing to change on screen. Also makes Typo: thing r=me
Timothy Hatcher
Comment 5 2008-09-28 16:31:49 PDT
Landed in r37046.
Note You need to log in before you can comment on or make changes to this bug.