diff options
author | Gokul Soumya | 2023-02-05 21:30:52 +0000 |
---|---|---|
committer | Blaž Hrastnik | 2023-02-11 07:05:23 +0000 |
commit | 937825e8fc415f2236c5effde7b5543f51fef542 (patch) | |
tree | c2c9183fa9367f1a1f691129f97ce5b70fd96dca | |
parent | 8e9c3299521cb886e1cd4e18f75ffd8f7b8723f9 (diff) |
Rename completion doc popup area variables
-rw-r--r-- | helix-term/src/ui/completion.rs | 29 |
1 files changed, 16 insertions, 13 deletions
diff --git a/helix-term/src/ui/completion.rs b/helix-term/src/ui/completion.rs index d5f001d1..6263106c 100644 --- a/helix-term/src/ui/completion.rs +++ b/helix-term/src/ui/completion.rs @@ -448,39 +448,42 @@ impl Component for Completion { let (popup_x, popup_y) = self.popup.get_rel_position(area, cx); let (popup_width, _popup_height) = self.popup.get_size(); - let mut width = area + let doc_width_available = area .width .saturating_sub(popup_x) .saturating_sub(popup_width); - let area = if width > 30 { - let mut height = area.height.saturating_sub(popup_y); + let doc_area = if doc_width_available > 30 { + let mut doc_width = doc_width_available; + let mut doc_height = area.height.saturating_sub(popup_y); let x = popup_x + popup_width; let y = popup_y; - if let Some((rel_width, rel_height)) = markdown_doc.required_size((width, height)) { - width = rel_width.min(width); - height = rel_height.min(height); + if let Some((rel_width, rel_height)) = + markdown_doc.required_size((doc_width, doc_height)) + { + doc_width = rel_width.min(doc_width); + doc_height = rel_height.min(doc_height); } - Rect::new(x, y, width, height) + Rect::new(x, y, doc_width, doc_height) } else { let half = area.height / 2; - let height = 15.min(half); + let doc_height = 15.min(half); // we want to make sure the cursor is visible (not hidden behind the documentation) let y = if cursor_pos + area.y - >= (cx.editor.tree.area().height - height - 2/* statusline + commandline */) + >= (cx.editor.tree.area().height - doc_height - 2/* statusline + commandline */) { 0 } else { // -2 to subtract command line + statusline. a bit of a hack, because of splits. - area.height.saturating_sub(height).saturating_sub(2) + area.height.saturating_sub(doc_height).saturating_sub(2) }; - Rect::new(0, y, area.width, height) + Rect::new(0, y, area.width, doc_height) }; // clear area let background = cx.editor.theme.get("ui.popup"); - surface.clear_with(area, background); - markdown_doc.render(area, surface, cx); + surface.clear_with(doc_area, background); + markdown_doc.render(doc_area, surface, cx); } } |