Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | test: undo moving point | Kartik K. Agaram | 2022-06-14 | 2 | -1/+47 | |
| | ||||||
* | test: undo naming a point | Kartik K. Agaram | 2022-06-14 | 3 | -1/+45 | |
| | | | | Also ensure we autosave. | |||||
* | test: autosave after name/move/delete of point | Kartik K. Agaram | 2022-06-14 | 2 | -3/+22 | |
| | ||||||
* | test: autosave after any shape | Kartik K. Agaram | 2022-06-14 | 4 | -4/+43 | |
| | ||||||
* | tests: deleting points/shapes | Kartik K. Agaram | 2022-06-14 | 1 | -1/+105 | |
| | ||||||
* | test: moving a point | Kartik K. Agaram | 2022-06-14 | 3 | -16/+44 | |
| | | | | | I found some code in the process that seems unreachable. Some chance of a regression here.. | |||||
* | test: naming a point | Kartik K. Agaram | 2022-06-14 | 3 | -2/+39 | |
| | ||||||
* | a little more prose describing manual_tests | Kartik K. Agaram | 2022-06-14 | 2 | -23/+30 | |
| | ||||||
* | one less manual test | Kartik K. Agaram | 2022-06-14 | 1 | -1/+0 | |
| | | | | Still a few more we can write. | |||||
* | drop non-existent feature from comment | Kartik K. Agaram | 2022-06-14 | 1 | -1/+0 | |
| | ||||||
* | tests for drawing polygons | Kartik K. Agaram | 2022-06-14 | 2 | -11/+180 | |
| | ||||||
* | more drawing tests | Kartik K. Agaram | 2022-06-14 | 2 | -1/+133 | |
| | ||||||
* | start writing some tests for drawings | Kartik K. Agaram | 2022-06-14 | 2 | -1/+33 | |
| | ||||||
* | . | Kartik K. Agaram | 2022-06-14 | 1 | -2/+2 | |
| | ||||||
* | extract variables for drawing padding | Kartik K. Agaram | 2022-06-14 | 3 | -6/+10 | |
| | ||||||
* | extract variable Margin_left | Kartik K. Agaram | 2022-06-14 | 4 | -86/+81 | |
| | ||||||
* | . | Kartik K. Agaram | 2022-06-14 | 1 | -1/+0 | |
| | ||||||
* | stop generating invalid keychords | Kartik K. Agaram | 2022-06-14 | 1 | -0/+1 | |
| | ||||||
* | some more manual tests related to drawings | Kartik K. Agaram | 2022-06-14 | 1 | -0/+5 | |
| | | | | | In the process I notice an issue: we seem to be adding no-op undo records after every drawing operation. | |||||
* | lots more tests | Kartik K. Agaram | 2022-06-14 | 2 | -32/+65 | |
| | | | | | | | I've been adding diligently to manual_tests but not actually _performing_ any manual tests before releases. They were just a todo list of automated tests to write, and long out of date. Now the list is up to date and much shorter. | |||||
* | mouse buttons are integers, not strings | Kartik K. Agaram | 2022-06-14 | 5 | -45/+45 | |
| | | | | | | Not sure where that idiom comes from or why strings work in some places (auto-coercion?). I picked it up off some example apps. But https://love2d.org/wiki/love.mouse.isDown says it should be an integer. | |||||
* | answered an open question | Kartik K. Agaram | 2022-06-14 | 1 | -1/+0 | |
| | | | | | The default font seems somehow different than the font created by newFont with just a font size. | |||||
* | go through and fix similar issues | Kartik K. Agaram | 2022-06-14 | 2 | -6/+19 | |
| | | | | All places where string.sub was being passed a _pos variable. | |||||
* | cleanup | Kartik K. Agaram | 2022-06-14 | 1 | -6/+0 | |
| | ||||||
* | bugfix | Kartik K. Agaram | 2022-06-14 | 3 | -4/+46 | |
| | | | | | | | | | | | | | | | | | | | | | | | manifestation: clicking past end of a long, wrapping line containing non-ASCII would cause the cursor to disappear rather than position past end of screen line. Hitting enter would then throw an assertion with the following stack trace: Error: text.lua:381: bad argument #2 to 'sub' (number expected, got nil) stack traceback: [love "boot.lua"]:345: in function <[love "boot.lua"]:341> [C]: in function 'sub' text.lua:381: in function 'insert_return' text.lua:179: in function 'keychord_pressed' main.lua:495: in function 'keychord_pressed' keychord.lua:10: in function <keychord.lua:5> app.lua:34: in function <app.lua:25> [C]: in function 'xpcall' cause: the click caused a call to Text.to_pos_on_line whose result was not on a UTF-8 character boundary. fix: make to_pos_on_line utf8-aware. | |||||
* | bugfix: UTF-8 in compute_fragments | Kartik K. Agaram | 2022-06-14 | 3 | -5/+27 | |
| | ||||||
* | . | Kartik K. Agaram | 2022-06-14 | 1 | -2/+2 | |
| | ||||||
* | new test | Kartik K. Agaram | 2022-06-12 | 2 | -0/+26 | |
| | | | | | For commit e4e12c77ad which fixed a regression caused by commit 24a0d162ef. | |||||
* | new test | Kartik K. Agaram | 2022-06-12 | 1 | -0/+12 | |
| | | | | | | | | | | For commit ff88a2a927 which fixed a regression caused by commit e51ce12969. I'm trying to provide enough guardrails for myself and future readers without causing a combinatorial explosion in tests. The previous commit was able to get more value out of existing tests, but this test feels necessary. And useful in general without reference to a specific bug. | |||||
* | override mouse state lookups in tests | Kartik K. Agaram | 2022-06-12 | 6 | -76/+113 | |
| | | | | | | | If I'd had this stuff in my test harness earlier, two recent commits would have failed tests and given me early warning: ff88238ff1 ff88a2a927 | |||||
* | delete all tests once they've executed | Kartik K. Agaram | 2022-06-12 | 1 | -0/+4 | |
| | ||||||
* | redundant check | Kartik K. Agaram | 2022-06-12 | 1 | -4/+2 | |
| | ||||||
* | fix a second BSOD in #4 :/ | Kartik K. Agaram | 2022-06-12 | 1 | -1/+1 | |
| | | | | I need more tests. | |||||
* | bugfix in commit e51ce12969 | Kartik K. Agaram | 2022-06-12 | 1 | -4/+6 | |
| | | | | | | | | | Any time I press a ctrl- chord LÖVE actually sees two key chords: C-lctrl C-... (the real one) But it's not just that. There's also a lot in the codebase that's just habit-based. I need more tests. | |||||
* | bugfix: BSOD in #4. | Kartik K. Agaram | 2022-06-11 | 2 | -4/+4 | |
| | | | | I messed up a function call in commit 391d764e13. | |||||
* | bugfix: autosave and undo in a couple of cases | Kartik K. Agaram | 2022-06-11 | 1 | -0/+6 | |
| | | | | This fixes part of #4, but not the BSOD. | |||||
* | bugfix: missed fixing a callsite | Kartik K. Agaram | 2022-06-11 | 1 | -1/+1 | |
| | ||||||
* | . | Kartik K. Agaram | 2022-06-11 | 1 | -1/+1 | |
| | ||||||
* | things seem to feel snappier now | Kartik K. Agaram | 2022-06-10 | 1 | -6/+2 | |
| | | | | | However, I think a lot of the benefit comes from just turning JIT off. Turning it on is still noticably sluggish. | |||||
* | stop handling nil screen_line_starting_pos everywhere | Kartik K. Agaram | 2022-06-10 | 2 | -24/+6 | |
| | | | | Things seem to be working.. | |||||
* | this fixes the immediate regression | Kartik K. Agaram | 2022-06-10 | 1 | -19/+17 | |
| | ||||||
* | stop repeatedly checking for line wrapping | Kartik K. Agaram | 2022-06-10 | 1 | -7/+8 | |
| | | | | | | We have a regression since we started reclaiming love Text fragments more aggressively in commit 69c5d844ccc. Pressing pageup no longer knows about any line's screen lines. Not fixed yet. | |||||
* | slight reorg in Readme | Kartik K. Agaram | 2022-06-10 | 1 | -2/+4 | |
| | ||||||
* | more precise scroll on paste | Kartik K. Agaram | 2022-06-10 | 1 | -19/+4 | |
| | ||||||
* | faster paste | Kartik K. Agaram | 2022-06-10 | 2 | -13/+11 | |
| | | | | | We don't need to perform the scroll calculations after inserting every single character from the clipboard. | |||||
* | revert previous commit | Kartik K. Agaram | 2022-06-10 | 1 | -5/+15 | |
| | ||||||
* | experiment: extremely precise scrolling on paste | Kartik K. Agaram | 2022-06-10 | 1 | -15/+5 | |
| | | | | But this is too slow. | |||||
* | clean up memory leak experiments | Kartik K. Agaram | 2022-06-10 | 2 | -517/+0 | |
| | ||||||
* | while we're at it, undo naming points | Kartik K. Agaram | 2022-06-10 | 1 | -0/+2 | |
| | ||||||
* | set current_drawing_index with current_drawing | Kartik K. Agaram | 2022-06-10 | 1 | -3/+6 | |
| |