Fix for #16771. #17698

Merged
merged 1 commit into from Jan 23, 2017

Projects

None yet

5 participants

@jmdowns2

Now insertCursorAtEndOfEachLineSelected works with multiple selections.

@msftclas

Hi @jmdowns2, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

+ let newSelections = new Array<ISelection>();
+ let selections = editor.getSelections();
+
+ for (var i = 0; i < selections.length; ++i) {
@yajiex
yajiex Dec 23, 2016 edited Member

This can be simplified to let newSelections = selections.map(selection => this.getCursorsForSelection(selection, editor));

Matt Downs Fix for #16771. Now insertCursorAtEndOfEachLineSelected works with mu…
…ltiple selections.
cabcd28
@alexandrudima alexandrudima merged commit c937836 into Microsoft:master Jan 23, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@alexandrudima alexandrudima added this to the January 2017 milestone Jan 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment