[CLOSED] Fix scroll position jumping for few cases

[core-ai-bot]

Issue by redmunds
Tuesday Oct 21, 2014 at 21:25 GMT
Originally opened as adobe/brackets#9624

---

This is for #9547 . This bug was exposed by fix for #9037.

May also help #9629

This change fixes these cases:

• Ctrl-Shift-O then click in page
• Ctrl-Shift-O then blur editor
• Ctrl-G, type in a line number, then Enter key
• Ctrl-G then blur editor
• Ctrl-T then blur editor
• Ctrl-G then click in page
• Ctrl-T then click in page

Verify that these case are not broken:

• Ctrl-Shift-O then Esc key
• Ctrl-Shift-O, select same file with arrow keys, then Enter key
• Ctrl-Shift-O, select different file with arrow keys, then Enter key
• Ctrl-G then Esc key
• Ctrl-T, select element, then Enter key
• Ctrl-T then Esc key

Note: "blur editor" can be done by selecting same file in working set

cc@peterflynn

---

redmunds included the following code: https://github.com/adobe/brackets/pull/9624/commits

[core-ai-bot]

Comment by redmunds
Monday Oct 27, 2014 at 21:02 GMT

---

Check if this also fixes #9629

[core-ai-bot]

Comment by redmunds
Thursday Dec 04, 2014 at 18:20 GMT

---

Removing DEVELOPMENT and PICK ME UP labels -- Waffle no longer seems to pick up pull requests.

@peterflynn You filed #9547, so do you care to review this one? This was originally tagged for 1.0, so I think we probably want to get this in for 1.1.

[core-ai-bot]

Comment by RaymondLim
Wednesday Dec 10, 2014 at 22:44 GMT

---

Looks good! Merging.