Last Comment Bug 785151 - Use ScrollIntoViewIfNeeded in the markup panel
: Use ScrollIntoViewIfNeeded in the markup panel
Status: RESOLVED FIXED
[fixed-in-fx-team]
:
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
:
Mentors:
Depends on:
Blocks:
  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:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


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

Description 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 Dave Camp (:dcamp) 2012-08-23 12:15:45 PDT
Created attachment 654729 [details] [diff] [review]
fix

ScrollIntoViewIfNeeded worked fine, I was just using the wrong element.
Comment 3 Dave Camp (:dcamp) 2012-08-25 17:02:13 PDT
https://hg.mozilla.org/mozilla-central/rev/73b1af99b72d

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