ZIm/crates/language_tools/src
Conrad Irwin a675ca7a1e
Remove into SelectionEffects from .change_selections (#33554)
In #32656 I generalized the argument to change selections to allow
controling both the scroll and the nav history (and the completion
trigger).

To avoid conflicting with ongoing debugger cherry-picks I left the
argument as an `impl Into<>`, but I think it's clearer to make callers
specify what they want here.

I converted a lot of `None` arguments to `SelectionEffects::no_scroll()`
to be exactly compatible; but I think many people used none as an "i
don't care" value in which case Default::default() might be more
appropraite

Closes #ISSUE

Release Notes:

- N/A
2025-06-27 14:31:31 -06:00
..
key_context_view.rs Rename debug: commands to dev: (#30675) 2025-05-14 11:15:27 +02:00
language_tools.rs Add language server control tool into the status bar (#32490) 2025-06-25 19:57:28 +03:00
lsp_log.rs lsp-log: Avoid trimming leading space in language server logs (#33418) 2025-06-25 16:24:51 -06:00
lsp_log_tests.rs zlog: Replace usages of env_logger in tests with zlog (#31436) 2025-05-26 11:48:50 -04:00
lsp_tool.rs Rework LSP tool keyboard story (#33525) 2025-06-27 16:25:56 +03:00
syntax_tree_view.rs Remove into SelectionEffects from .change_selections (#33554) 2025-06-27 14:31:31 -06:00