Fix helix-editor#6605: Remove soft-wrap.enable option wrapping.

gibbz00 · Apr 8, 2023 · 1103d39 · 1103d39
1 parent e856906
commit 1103d39
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/helix-core/src/syntax.rs b/helix-core/src/syntax.rs
@@ -555,7 +555,7 @@ impl LanguageConfiguration {
 #[serde(default, rename_all = "kebab-case", deny_unknown_fields)]
 pub struct SoftWrap {
     /// Soft wrap lines that exceed viewport width. Default to off
-    pub enable: Option<bool>,
+    pub enable: bool,
     /// Maximum space left free at the end of the line.
     /// This space is used to wrap text at word boundaries. If that is not possible within this limit
     /// the word is simply split at the end of the line.

diff --git a/helix-view/src/document.rs b/helix-view/src/document.rs
@@ -1444,9 +1444,8 @@ impl Document {
             .as_ref()
             .and_then(|config| config.soft_wrap.as_ref());
         let enable_soft_wrap = language_soft_wrap
-            .and_then(|soft_wrap| soft_wrap.enable)
-            .or(editor_soft_wrap.enable)
-            .unwrap_or(false);
+            .map(|soft_wrap| soft_wrap.enable)
+            .unwrap_or_else(|| editor_soft_wrap.enable);
         let max_wrap = language_soft_wrap
             .and_then(|soft_wrap| soft_wrap.max_wrap)
             .or(config.soft_wrap.max_wrap)