feat: add `:toggle-diagnostics` command to hide diagnostics

helix-term/src/commands/typed.rs

@@ -1940,8 +1940,8 @@ fn toggle_diagnostics(
     ensure!(args.is_empty(), ":toggle-diagnostics takes no arguments");
 
     let (view, _) = current!(cx.editor);
-
+    view.diagnostics_handler.toggle_active();
-    view.diagnostics_handler.toggle_diagnostics();
+    cx.editor.toggle_diagnostics();
 
     Ok(())
 }

helix-term/src/ui/editor.rs

@@ -189,19 +189,26 @@ impl EditorView {
                 primary_cursor,
             });
         }
-        let width = view.inner_width(doc);
         let config = doc.config.load();
-        let enable_cursor_line = view
+
-            .diagnostics_handler
+        if editor.show_diagnostics {
-            .show_cursorline_diagnostics(doc, view.id);
+            log::error!("{:#?}", editor.show_diagnostics);
-        let inline_diagnostic_config = config.inline_diagnostics.prepare(width, enable_cursor_line);
+            let width = view.inner_width(doc);
-        decorations.add_decoration(InlineDiagnostics::new(
+            let enable_cursor_line = view
-            doc,
+                .diagnostics_handler
-            theme,
+                .show_cursorline_diagnostics(doc, view.id);
-            primary_cursor,
+            let inline_diagnostic_config =
-            inline_diagnostic_config,
+                config.inline_diagnostics.prepare(width, enable_cursor_line);
-            config.end_of_line_diagnostics,
+
-        ));
+            decorations.add_decoration(InlineDiagnostics::new(
+                doc,
+                theme,
+                primary_cursor,
+                inline_diagnostic_config,
+                config.end_of_line_diagnostics,
+            ));
+        }
+
         render_document(
             surface,
             inner,
@@ -229,6 +236,7 @@ impl EditorView {
 
         if config.inline_diagnostics.disabled()
             && config.end_of_line_diagnostics == DiagnosticFilter::Disable
+            && editor.show_diagnostics
         {
             Self::render_diagnostics(doc, view, inner, surface, theme);
         }

helix-term/src/ui/text_decorations/diagnostics.rs

@@ -123,6 +123,7 @@ impl Renderer<'_, '_> {
         end_col - start_col
     }
 
+    // need to toggle this
     fn draw_diagnostic(&mut self, diag: &Diagnostic, col: u16, next_severity: Option<Severity>) {
         let severity = diag.severity();
         let (sym, sym_severity) = if let Some(next_severity) = next_severity {

helix-view/src/handlers/diagnostics.rs

@@ -94,9 +94,6 @@ impl DiagnosticsHandler {
 }
 
 impl DiagnosticsHandler {
-    pub fn toggle_diagnostics(&mut self) {
-        self.active = !self.active;
-    }
     pub fn immediately_show_diagnostic(&self, doc: &Document, view: ViewId) {
         self.last_doc.set(doc.id());
         let cursor_line = doc
@@ -107,6 +104,11 @@ impl DiagnosticsHandler {
         self.active_generation
             .store(self.generation.get(), atomic::Ordering::Relaxed);
     }
+
+    pub fn toggle_active(&mut self) {
+        self.active = !self.active;
+    }
+
     pub fn show_cursorline_diagnostics(&self, doc: &Document, view: ViewId) -> bool {
         if !self.active {
             return false;