SignatureHelp: bugfix: introduce minimum width of 10

required_size shouldn't return None. Introduce minimum width so that the height can be calculated Signed-off-by: Ben Fekih, Hichem <[email protected]>
helix-editor · Apr 19, 2024 · 3577335 · 3577335
1 parent 94405f3
commit 3577335
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/helix-term/src/ui/lsp.rs b/helix-term/src/ui/lsp.rs
@@ -155,10 +155,8 @@ impl Component for SignatureHelp {
 
         let sig = &self.signatures[self.active_signature];
 
-        if PADDING >= viewport.1 || PADDING >= viewport.0 {
-            return None;
-        }
-        let max_text_width = (viewport.0 - PADDING).min(120);
+        // Min width: 10, Max width: 120
+        let max_text_width = viewport.0.saturating_sub(PADDING).min(120).max(10);
 
         let signature_text = crate::ui::markdown::highlighted_code_block(
             sig.signature.as_str(),