diff --git a/crates/editor/src/editor.rs b/crates/editor/src/editor.rs index 6e5543132c8e7892ad09666e6a733eea535ce634..730482b123bf77f9691cf7a409885de4644b1e3c 100644 --- a/crates/editor/src/editor.rs +++ b/crates/editor/src/editor.rs @@ -3442,7 +3442,7 @@ impl Editor { s.select(new_selections) }); - if !bracket_inserted && EditorSettings::get_global(cx).use_on_type_format { + if !bracket_inserted { if let Some(on_type_format_task) = this.trigger_on_type_formatting(text.to_string(), cx) { @@ -4191,6 +4191,15 @@ impl Editor { .read(cx) .text_anchor_for_position(position, cx)?; + let settings = language_settings::language_settings( + buffer.read(cx).language_at(buffer_position).as_ref(), + buffer.read(cx).file(), + cx, + ); + if !settings.use_on_type_format { + return None; + } + // OnTypeFormatting returns a list of edits, no need to pass them between Zed instances, // hence we do LSP request & edit on host side only — add formats to host's history. let push_to_lsp_host_history = true; diff --git a/crates/editor/src/editor_settings.rs b/crates/editor/src/editor_settings.rs index d651e76c2c2e7d60ccf45316cdc725c31e6a3409..9137629241468c826582fbc9f133552b5ca0fe9d 100644 --- a/crates/editor/src/editor_settings.rs +++ b/crates/editor/src/editor_settings.rs @@ -13,7 +13,6 @@ pub struct EditorSettings { pub show_completions_on_input: bool, pub show_completion_documentation: bool, pub completion_documentation_secondary_query_debounce: u64, - pub use_on_type_format: bool, pub toolbar: Toolbar, pub scrollbar: Scrollbar, pub gutter: Gutter, @@ -209,11 +208,6 @@ pub struct EditorSettingsContent { /// /// Default: 300 ms pub completion_documentation_secondary_query_debounce: Option, - /// Whether to use additional LSP queries to format (and amend) the code after - /// every "trigger" symbol input, defined by LSP server capabilities. - /// - /// Default: true - pub use_on_type_format: Option, /// Toolbar related settings pub toolbar: Option, /// Scrollbar related settings diff --git a/crates/language/src/language_settings.rs b/crates/language/src/language_settings.rs index 735a9a60f87fa614048002c7f121054cb9a98c96..f830c5f25c308cbe5a22145b185f5c5c2a8ffd4d 100644 --- a/crates/language/src/language_settings.rs +++ b/crates/language/src/language_settings.rs @@ -113,6 +113,9 @@ pub struct LanguageSettings { pub use_autoclose: bool, /// Whether to automatically surround text with brackets. pub use_auto_surround: bool, + /// Whether to use additional LSP queries to format (and amend) the code after + /// every "trigger" symbol input, defined by LSP server capabilities. + pub use_on_type_format: bool, // Controls how the editor handles the autoclosed characters. pub always_treat_brackets_as_autoclosed: bool, /// Which code actions to run on save @@ -333,6 +336,11 @@ pub struct LanguageSettingsContent { /// /// Default: false pub always_treat_brackets_as_autoclosed: Option, + /// Whether to use additional LSP queries to format (and amend) the code after + /// every "trigger" symbol input, defined by LSP server capabilities. + /// + /// Default: true + pub use_on_type_format: Option, /// Which code actions to run on save after the formatter. /// These are not run if formatting is off. /// @@ -1045,6 +1053,7 @@ fn merge_settings(settings: &mut LanguageSettings, src: &LanguageSettingsContent merge(&mut settings.soft_wrap, src.soft_wrap); merge(&mut settings.use_autoclose, src.use_autoclose); merge(&mut settings.use_auto_surround, src.use_auto_surround); + merge(&mut settings.use_on_type_format, src.use_on_type_format); merge( &mut settings.always_treat_brackets_as_autoclosed, src.always_treat_brackets_as_autoclosed,