akkartik/pong.love - Change COMBEL3F5MWESKCIKONPJE6VKEKXHMC3J2777EBJLQ47WE2O2XIQC

clicking on side B now works even past line wrap

It's getting pretty clear that Text.screen_line_indexB doesn't do the right thing when startposB is nil. I'm still not sure, though, what the right API(s) should be.

Created by Kartik K. Agaram on August 18, 2022

COMBEL3F5MWESKCIKONPJE6VKEKXHMC3J2777EBJLQ47WE2O2XIQC

Dependencies

In channels

main

Change contents

Replacement in text.lua at line 950 [5.1]

∅:D[3.638] → [4.2961:3111]

B:BD[4.2961] → [4.2961:3111]

  assert(line_cache.startposB)
  local start_screen_line_indexB = Text.screen_line_indexB(line_cache.screen_line_starting_posB, line_cache.startposB)

[3.638]

[4.3111]

  local start_screen_line_indexB
  if line_cache.startposB then
    start_screen_line_indexB = Text.screen_line_indexB(line_cache.screen_line_starting_posB, line_cache.startposB)
  else
    start_screen_line_indexB = 2  -- skip the first line of side B, which we checked above
  end

Deletion in edit.lua at line 143 [7.413]
B:BD[6.4696] → [2.1213:1247]
```
    print('checking', line_index)
```

Deletion in edit.lua at line 144 [7.413]

B:BD[8.9966] → [2.1248:1384]

      print('in line', #State.line_cache[line_index].screen_line_starting_pos, #State.line_cache[line_index].screen_line_starting_posB)