Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | document one more shortcut | Kartik K. Agaram | 2022-06-17 | 1 | -0/+1 |
| | |||||
* | tweak to Readme | Kartik K. Agaram | 2022-06-15 | 1 | -2/+2 |
| | |||||
* | new fork for #1 | Kartik K. Agaram | 2022-06-15 | 1 | -0/+4 |
| | |||||
* | bugfix: alignment of help screen | Kartik K. Agaram | 2022-06-15 | 1 | -33/+33 |
| | |||||
* | dead code | Kartik K. Agaram | 2022-06-15 | 1 | -45/+0 |
| | |||||
* | move | Kartik K. Agaram | 2022-06-15 | 1 | -28/+28 |
| | |||||
* | . | Kartik K. Agaram | 2022-06-15 | 1 | -6/+3 |
| | |||||
* | stop recording points for arcs | Kartik K. Agaram | 2022-06-15 | 2 | -2/+1 |
| | | | | | | It was kinda weird that we were recording the start but not the end. And moving the start point didn't actually affect the arc. Let's see if we actually ever need it. | ||||
* | move current mode indicator slightly | Kartik K. Agaram | 2022-06-15 | 1 | -2/+2 |
| | |||||
* | standardize on ordering of cases | Kartik K. Agaram | 2022-06-15 | 1 | -9/+9 |
| | |||||
* | . | Kartik K. Agaram | 2022-06-14 | 1 | -3/+3 |
| | |||||
* | all pending manual tests done! | Kartik K. Agaram | 2022-06-14 | 2 | -7/+39 |
| | |||||
* | 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 |
| |