update internal cursor position per Code selection change

[rebornix]

Yay! We love PRs! 🎊

Please include a description of your change and ensure:

• Commit message has a short title & issue references
• Each commit does a logical chunk of work.
• It builds and tests pass (e.g gulp)

More info can be found on our contribution guide.

[xconverge]

Does this fix a problem and then create all of the original problems that we closed before we made this change? It makes sense to me, but I imagine some of the issues closed from the other day will reopen.

[rebornix]

let me give them a quick check.

[rebornix]

Verified #872 #877 #882 #892 #895 #896, they are still good.

Verified #856, which we claimed it was fixed while actually it was still half broken without this PR.

#901 I'm not sure if I'm doing the right keystrokes, on both Master and my PR, it still doesn't work.

[xconverge]

LGTM then, thanks for checking those

What portion of the code was causing #882 then...

[johnfn]

@xconverge I think the problem was that it would always go into that complex selection logic and eventually trigger updateView. Now that happens much more rarely.

So I think we're good here.