diff options
author | Kartik K. Agaram <vc@akkartik.com> | 2024-02-08 02:23:56 -0800 |
---|---|---|
committer | Kartik K. Agaram <vc@akkartik.com> | 2024-02-08 02:37:12 -0800 |
commit | 4e9298dda1fb67527d5fdd3172217b803b27e652 (patch) | |
tree | f14402956377fd275e2b9033bb88ad314d8c3013 /source_text.lua | |
parent | 1dbd734abb757c924d2f72b5eaa6663712962035 (diff) | |
download | lines.love-4e9298dda1fb67527d5fdd3172217b803b27e652.tar.gz |
bugfix in cursor positioning
scenario: - create a long wrapping line - tap past end of first screen line Before this commit the cursor would be positioned not quite at the end of the screen line but one character before. In effect there was no way to position cursor at end of a wrapping line. I'm not sure how this bug has lasted so long. It was introduced in commit 8d3adfa36 back in June 2022, which was itself billed as a bugfix for "clicking past end of screen line". But when I go back to it this bug exists even back then. How did I miss it?! I wrote a test back then -- and the test was wrong, has always been wrong.
Diffstat (limited to 'source_text.lua')
-rw-r--r-- | source_text.lua | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source_text.lua b/source_text.lua index b22d358..7c1838c 100644 --- a/source_text.lua +++ b/source_text.lua @@ -798,7 +798,7 @@ function Text.to_pos_on_line(State, line_index, mx, my) -- (The final screen line positions past end of screen line as always.) if screen_line_index < #line_cache.screen_line_starting_pos and mx > State.left + Text.screen_line_width(State, line_index, screen_line_index) then --? print('past end of non-final line; return') - return line_cache.screen_line_starting_pos[screen_line_index+1]-1 + return line_cache.screen_line_starting_pos[screen_line_index+1] end local s = string.sub(line.data, screen_line_starting_byte_offset) --? print('return', mx, Text.nearest_cursor_pos(State.font, s, mx, State.left), '=>', screen_line_starting_pos + Text.nearest_cursor_pos(State.font, s, mx, State.left) - 1) |