-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Port item up down movers from IntelliJ #6823
Comments
Alternatively could we do this with a command? |
Yeah, that would need to be a command in VS Code, and a request-response in LSP. The user can then bind the command to ctrl+shift+up/down or whatever |
I think the LSP side could be a command as well to avoid another custom request/response. |
Ah, sorry, I've misunderstood. Your idea is that the client will send If that's right, I'd rather do a custom extension still, for two reasons:
|
Yes that was the idea. You would still need custom code to surface the command of course but the benefit to other clients is that they wouldn't have to write as much. |
Intellij Rust has this bunch of code:
https://github.com/intellij-rust/intellij-rust/tree/25f37175094192ba88f345f03b2a4b5b941dd395/src/main/kotlin/org/rust/ide/actions/mover
this is for moving items, statements and mactch arms up and down. rust-analyzer should have this as well, although I don't think that this is exposed in the procol. Rough steps:
lsp-extensions.md
)ide
crate. This only needs access tosyntax
, so should be rather trivial.The text was updated successfully, but these errors were encountered: