From 4dbc23ff1ca0c3cc601c6ea011d4ec0a294dd7b2 Mon Sep 17 00:00:00 2001 From: wojciechkepka Date: Wed, 9 Jun 2021 18:21:58 +0200 Subject: [PATCH] Fix documentation popup panic --- helix-term/src/ui/popup.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/helix-term/src/ui/popup.rs b/helix-term/src/ui/popup.rs index 015d5c9b5..ca349403a 100644 --- a/helix-term/src/ui/popup.rs +++ b/helix-term/src/ui/popup.rs @@ -125,13 +125,13 @@ impl Component for Popup { let mut rel_x = position.col as u16; let mut rel_y = position.row as u16; if viewport.width <= rel_x + width { - rel_x -= ((rel_x + width) - viewport.width) + rel_x = rel_x.saturating_sub((rel_x + width).saturating_sub(viewport.width)); }; // TODO: be able to specify orientation preference. We want above for most popups, below // for menus/autocomplete. if height <= rel_y { - rel_y -= height // position above point + rel_y = rel_y.saturating_sub(height) // position above point } else { rel_y += 1 // position below point }