akkartik/driver.love - Change SY5SNGKD3JUF3S6Q3RVOWEXMKOEQUXFKFRLMSLBPE6RN4YU7DAZQC

let's see if this is easier to understand in 6 mos

I spent 4 days agonizing over a bug in driver.love, working up the courage to think about it, and then a whole day trying to make sense of scrolling and deeply questioning my right to tell anyone anything about programming.

Created by Kartik K. Agaram on October 22, 2023

SY5SNGKD3JUF3S6Q3RVOWEXMKOEQUXFKFRLMSLBPE6RN4YU7DAZQC

Dependencies

[2] AUZQ7ULP65IGCJYIT3U6R42HEJT35R33TQHA7XNK242KWCV6IKMAC
[3] QJAYOFWY7V4BWVLJMEDCCBC2HX4BDAZI6PQVLWCLYRHWGMOBFTVAC
[4] P36KIMZCWGBG6CZQE3W5SVTCJMYZOVEZ3U3FAF46SO5ZMMM42PHQC

In channels

main

Change contents

Replacement in 0030-update_editor_box at line 3 [3.26]

B:BD[3.137] → [4.27:153]

	-- Compute screen_top1 in viewport coordinates because the editor's font takes scaling into account.
	if vy(node.y) > 0 then

[3.137]

[2.67]

	if node.y > Viewport.y then

Replacement in 0030-update_editor_box at line 10 [3.26]

B:BD[3.352] → [4.154:238]

		node.editor.screen_top1, node.editor.top = schema1_of_y(node.editor, -vy(node.y))

[3.352]

[4.238]

		node.editor.screen_top1, node.editor.top = schema1_of_y(node.editor, scale(Viewport.y-node.y))  -- scale y because editor's font is scaled