akkartik/sokoban.love - Change SWZAQHGRX3T5MZVM7YZ4EACZX5ON2NUZG5YXLWVEXLL3WZUZYAFQC

bugfix: up arrow when line above is a drawing

This bug was introduced in commit 528c64d690 on 2022-09-05 :/

Created by Kartik K. Agaram on February 1, 2023

SWZAQHGRX3T5MZVM7YZ4EACZX5ON2NUZG5YXLWVEXLL3WZUZYAFQC

Dependencies

In channels

main

Change contents

Insertion in text_tests.lua at line 1056 [3.2]

[2.9436]

[3.11325]

end
function test_down_arrow_skips_drawing()
  -- some lines of text with a drawing intermixed
  local drawing_width = 50
  App.screen.init{width=Editor_state.left+drawing_width, height=100}
  Editor_state = edit.initialize_test_state()
  Editor_state.lines = load_array{'abc',               -- height 15
                                  '```lines', '```',   -- height 25
                                  'ghi'}
  Text.redraw_all(Editor_state)
  Editor_state.cursor1 = {line=1, pos=1}
  Editor_state.screen_top1 = {line=1, pos=1}
  Editor_state.screen_bottom1 = {}
  edit.draw(Editor_state)
  local y = Editor_state.top
  App.screen.check(y, 'abc', 'baseline/screen:1')
  y = y + Editor_state.line_height
  local drawing_height = Drawing_padding_height + drawing_width/2  -- default
  y = y + drawing_height
  App.screen.check(y, 'ghi', 'baseline/screen:3')
  check(Editor_state.cursor_x, 'baseline/cursor_x')
  -- after hitting the down arrow the cursor moves down by 2 lines, skipping the drawing
  edit.run_after_keychord(Editor_state, 'down')
  check_eq(Editor_state.cursor1.line, 3, 'cursor')

Insertion in text_tests.lua at line 1231 [3.2]

[3.19428]

function test_up_arrow_skips_drawing()
  -- some lines of text with a drawing intermixed
  local drawing_width = 50
  App.screen.init{width=Editor_state.left+drawing_width, height=100}
  Editor_state = edit.initialize_test_state()
  Editor_state.lines = load_array{'abc',               -- height 15
                                  '```lines', '```',   -- height 25
                                  'ghi'}
  Text.redraw_all(Editor_state)
  Editor_state.cursor1 = {line=3, pos=1}
  Editor_state.screen_top1 = {line=1, pos=1}
  Editor_state.screen_bottom1 = {}
  edit.draw(Editor_state)
  local y = Editor_state.top
  App.screen.check(y, 'abc', 'baseline/screen:1')
  y = y + Editor_state.line_height
  local drawing_height = Drawing_padding_height + drawing_width/2  -- default
  y = y + drawing_height
  App.screen.check(y, 'ghi', 'baseline/screen:3')
  check(Editor_state.cursor_x, 'baseline/cursor_x')
  -- after hitting the up arrow the cursor moves up by 2 lines, skipping the drawing
  edit.run_after_keychord(Editor_state, 'up')
  check_eq(Editor_state.cursor1.line, 1, 'cursor')
end

Replacement in text.lua at line 429 [6.1]

B:BD[4.657] → [5.415:476]

        State.cursor1 = {line=State.cursor1.line-1, pos=nil}

[4.657]

[7.208]

        State.cursor1 = {line=new_cursor_line, pos=nil}

Insertion in source_text_tests.lua at line 1026 [8.3498]

[2.32282]

[8.38177]

end
function test_down_arrow_skips_drawing()
  -- some lines of text with a drawing intermixed
  local drawing_width = 50
  App.screen.init{width=Editor_state.left+drawing_width, height=100}
  Editor_state = edit.initialize_test_state()
  Editor_state.lines = load_array{'abc',               -- height 15
                                  '```lines', '```',   -- height 25
                                  'ghi'}
  Text.redraw_all(Editor_state)
  Editor_state.cursor1 = {line=1, pos=1}
  Editor_state.screen_top1 = {line=1, pos=1}
  Editor_state.screen_bottom1 = {}
  edit.draw(Editor_state)
  local y = Editor_state.top
  App.screen.check(y, 'abc', 'baseline/screen:1')
  y = y + Editor_state.line_height
  local drawing_height = Drawing_padding_height + drawing_width/2  -- default
  y = y + drawing_height
  App.screen.check(y, 'ghi', 'baseline/screen:3')
  check(Editor_state.cursor_x, 'baseline/cursor_x')
  -- after hitting the down arrow the cursor moves down by 2 lines, skipping the drawing
  edit.run_after_keychord(Editor_state, 'down')
  check_eq(Editor_state.cursor1.line, 3, 'cursor')

Insertion in source_text_tests.lua at line 1201 [8.3498]

[8.47509]

function test_up_arrow_skips_drawing()
  -- some lines of text with a drawing intermixed
  local drawing_width = 50
  App.screen.init{width=Editor_state.left+drawing_width, height=100}
  Editor_state = edit.initialize_test_state()
  Editor_state.lines = load_array{'abc',               -- height 15
                                  '```lines', '```',   -- height 25
                                  'ghi'}
  Text.redraw_all(Editor_state)
  Editor_state.cursor1 = {line=3, pos=1}
  Editor_state.screen_top1 = {line=1, pos=1}
  Editor_state.screen_bottom1 = {}
  edit.draw(Editor_state)
  local y = Editor_state.top
  App.screen.check(y, 'abc', 'baseline/screen:1')
  y = y + Editor_state.line_height
  local drawing_height = Drawing_padding_height + drawing_width/2  -- default
  y = y + drawing_height
  App.screen.check(y, 'ghi', 'baseline/screen:3')
  check(Editor_state.cursor_x, 'baseline/cursor_x')
  -- after hitting the up arrow the cursor moves up by 2 lines, skipping the drawing
  edit.run_after_keychord(Editor_state, 'up')
  check_eq(Editor_state.cursor1.line, 1, 'cursor')
end

Replacement in source_text.lua at line 683 [8.83723]

B:BD[5.7174] → [5.7174:7235]

        State.cursor1 = {line=State.cursor1.line-1, pos=nil}

[5.7174]

[5.7235]

        State.cursor1 = {line=new_cursor_line, pos=nil}

Replacement in source_text.lua at line 722 [8.83723]

B:BD[5.8200] → [5.8200:8262]

        State.cursor1 = {line=State.cursor1.line-1, posB=nil}

[5.8200]

[5.8262]

        State.cursor1 = {line=new_cursor_line, posB=nil}