-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Select in Visual mode when scrolling #1591
Conversation
We don't merge this before Code ships. |
Ah, awesome! Thanks for the vscode-side work, @rebornix! |
Does this mean using the scrollwheel will expand / contract the selection? |
Can you bring this one up to date so that it is ready to be merged as well |
I think this also fixes #885 |
@rebornix can you rebase/update this one? |
@rebornix You want me to finish this one? |
I'll do this, sorry for being late
…On Tue, May 23, 2017 at 23:49 Horace He ***@***.***> wrote:
@rebornix <https://github.com/rebornix> You want me to finish this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1591 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AA1heHIO7Pbl34lvHgZdHA7avRCapCtcks5r87a0gaJpZM4NKtgJ>
.
|
This doesn't work once you have made it to the start or end of the pane, try to do it a few times and you will see it gets stuck |
@xconverge This is due to something on our end. I'm guessing that we're not updating cursor position properly. |
Closed in favor of this PR: #1859 |
Ref microsoft/vscode#22147 (comment) and fixes #907
With next version of Code, we can expand selection while scrolling.