fix: text_pos_at_screen_coords tests

3 years ago · d62ad8b595
parent fd02d1bf89
commit d62ad8b595
1 changed files with 8 additions and 4 deletions
--- 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!(