From 075bab2ea9d3b6b2199947c889fdce5648bcbac2 Mon Sep 17 00:00:00 2001 From: Kirill Bulatov Date: Fri, 5 May 2023 14:30:16 +0300 Subject: [PATCH] Use more convntional name for the settings --- assets/settings/default.json | 2 +- crates/editor/src/element.rs | 14 +++++++------- crates/settings/src/settings.rs | 12 ++++++------ 3 files changed, 14 insertions(+), 14 deletions(-) diff --git a/assets/settings/default.json b/assets/settings/default.json index f8d7406a9c..20d07acae0 100644 --- a/assets/settings/default.json +++ b/assets/settings/default.json @@ -42,7 +42,7 @@ // "none" // 3. Draw all invisible symbols: // "all" - "show_invisibles": "selection", + "show_whitespaces": "selection", // Whether the screen sharing icon is shown in the os status bar. "show_call_status_icon": true, // Whether to use language servers to provide code intelligence. diff --git a/crates/editor/src/element.rs b/crates/editor/src/element.rs index dc448aa17f..48dd33f3e9 100644 --- a/crates/editor/src/element.rs +++ b/crates/editor/src/element.rs @@ -37,7 +37,7 @@ use itertools::Itertools; use json::json; use language::{Bias, CursorShape, DiagnosticSeverity, OffsetUtf16, Selection}; use project::ProjectPath; -use settings::{GitGutter, Settings, ShowInvisibles}; +use settings::{GitGutter, Settings, ShowWhitespaces}; use smallvec::SmallVec; use std::{ borrow::Cow, @@ -1657,13 +1657,13 @@ fn draw_invisibles( let settings = cx.global::(); let regions_to_hit = match settings .editor_overrides - .show_invisibles - .or(settings.editor_defaults.show_invisibles) + .show_whitespaces + .or(settings.editor_defaults.show_whitespaces) .unwrap_or_default() { - ShowInvisibles::None => return, - ShowInvisibles::Selection => Some(selection_ranges), - ShowInvisibles::All => None, + ShowWhitespaces::None => return, + ShowWhitespaces::Selection => Some(selection_ranges), + ShowWhitespaces::All => None, }; for invisible in &line_with_invisibles.invisibles { @@ -2895,7 +2895,7 @@ mod tests { cx.update(|cx| { let mut test_settings = Settings::test(cx); - test_settings.editor_defaults.show_invisibles = Some(ShowInvisibles::All); + test_settings.editor_defaults.show_whitespaces = Some(ShowWhitespaces::All); test_settings.editor_defaults.tab_size = Some(NonZeroU32::new(tab_size).unwrap()); cx.set_global(test_settings); }); diff --git a/crates/settings/src/settings.rs b/crates/settings/src/settings.rs index 2e5fd1353a..7ba52fcb5e 100644 --- a/crates/settings/src/settings.rs +++ b/crates/settings/src/settings.rs @@ -173,7 +173,7 @@ pub struct EditorSettings { pub formatter: Option, pub enable_language_server: Option, pub show_copilot_suggestions: Option, - pub show_invisibles: Option, + pub show_whitespaces: Option, } #[derive(Copy, Clone, Debug, Serialize, Deserialize, PartialEq, Eq, JsonSchema)] @@ -449,7 +449,7 @@ pub struct FeaturesContent { #[derive(Copy, Clone, Debug, Default, Serialize, Deserialize, PartialEq, Eq, JsonSchema)] #[serde(rename_all = "snake_case")] -pub enum ShowInvisibles { +pub enum ShowWhitespaces { #[default] Selection, None, @@ -517,7 +517,7 @@ impl Settings { formatter: required(defaults.editor.formatter), enable_language_server: required(defaults.editor.enable_language_server), show_copilot_suggestions: required(defaults.editor.show_copilot_suggestions), - show_invisibles: required(defaults.editor.show_invisibles), + show_whitespaces: required(defaults.editor.show_whitespaces), }, editor_overrides: Default::default(), copilot: CopilotSettings { @@ -668,8 +668,8 @@ impl Settings { self.language_setting(language, |settings| settings.tab_size) } - pub fn show_invisibles(&self, language: Option<&str>) -> ShowInvisibles { - self.language_setting(language, |settings| settings.show_invisibles) + pub fn show_whitespaces(&self, language: Option<&str>) -> ShowWhitespaces { + self.language_setting(language, |settings| settings.show_whitespaces) } pub fn hard_tabs(&self, language: Option<&str>) -> bool { @@ -808,7 +808,7 @@ impl Settings { formatter: Some(Formatter::LanguageServer), enable_language_server: Some(true), show_copilot_suggestions: Some(true), - show_invisibles: Some(ShowInvisibles::None), + show_whitespaces: Some(ShowWhitespaces::None), }, editor_overrides: Default::default(), copilot: Default::default(),