From d62ad8b595a4f901b9c5dba1bb6e8f70ece395bf Mon Sep 17 00:00:00 2001 From: Blaž Hrastnik Date: Fri, 4 Mar 2022 11:23:05 +0900 Subject: fix: text_pos_at_screen_coords tests --- helix-view/src/view.rs | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/helix-view/src/view.rs b/helix-view/src/view.rs index b427a8d4..c6ae0c56 100644 --- a/helix-view/src/view.rs +++ b/helix-view/src/view.rs @@ -260,7 +260,7 @@ impl View { // If pos is in the middle of a wider grapheme (tab for example) // return the starting offset. - if col + width >= target { + if col + width > target { break; } @@ -356,7 +356,7 @@ mod tests { assert_eq!( view.text_pos_at_screen_coords(&text, 41, 40 + OFFSET + 1, 4), - Some(5) + Some(4) ); assert_eq!( @@ -384,9 +384,13 @@ mod tests { Some(0) ); + assert_eq!( + view.text_pos_at_screen_coords(&text, 40, 40 + OFFSET + 4, 4), + Some(4) + ); assert_eq!( view.text_pos_at_screen_coords(&text, 40, 40 + OFFSET + 5, 4), - Some(5) + Some(4) ); assert_eq!( @@ -396,7 +400,7 @@ mod tests { assert_eq!( view.text_pos_at_screen_coords(&text, 40, 40 + OFFSET + 7, 4), - Some(6) + Some(5) ); assert_eq!( -- cgit v1.2.3-70-g09d2