Last Comment Bug 785151 - Use ScrollIntoViewIfNeeded in the markup panel
: Use ScrollIntoViewIfNeeded in the markup panel
Product: Firefox
Classification: Client Software
Component: Developer Tools: Inspector (show other bugs)
: 13 Branch
: x86 Mac OS X
P2 normal (vote)
: Firefox 17
Assigned To: Nobody; OK to take it and work on it
: Gabriel Luong [:gl][1 biz day review guarantee] (ΦωΦ)
Depends on:
  Show dependency treegraph
Reported: 2012-08-23 11:09 PDT by Dave Camp (:dcamp)
Modified: 2012-08-25 17:02 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

fix (896 bytes, patch)
2012-08-23 12:15 PDT, Dave Camp (:dcamp)
jwalker: review+
Details | Diff | Splinter Review

Description User image Dave Camp (:dcamp) 2012-08-23 11:09:06 PDT
The markup panel doesn't currently scroll the selection into view.  Using ScrollIntoViewIfNeeded seemed to make the whole thing jumpy, this should be investigated.
Comment 1 User image Dave Camp (:dcamp) 2012-08-23 12:15:45 PDT
Created attachment 654729 [details] [diff] [review]

ScrollIntoViewIfNeeded worked fine, I was just using the wrong element.
Comment 3 User image Dave Camp (:dcamp) 2012-08-25 17:02:13 PDT

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