Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | move drawing.starty into line cache | Kartik K. Agaram | 2022-07-20 | 3 | -60/+71 |
| | |||||
* | use line cache for drawings as well | Kartik K. Agaram | 2022-07-20 | 3 | -31/+31 |
| | |||||
* | bugfix: where cursor is drawn | Kartik K. Agaram | 2022-07-20 | 3 | -14/+25 |
| | | | | | | | The published version of lines.love was broken for almost an hour. The cursor would render one position to the right of where it really is. To fix it, this commit rolls back 26ba6e4e5a71. There doesn't seem a good way to test it. | ||||
* | reorg manual tests doc | Kartik K. Agaram | 2022-07-20 | 1 | -9/+7 |
| | |||||
* | . | Kartik K. Agaram | 2022-07-20 | 1 | -0/+2 |
| | |||||
* | exclude left margin from my word-split heuristic | Kartik K. Agaram | 2022-07-20 | 2 | -3/+3 |
| | | | | | Gratifying how few tests need changing. Recent commits seem on the right track. | ||||
* | allow Text.nearest_pos_less_than to return 0 | Kartik K. Agaram | 2022-07-20 | 2 | -7/+4 |
| | | | | This eliminates another case of overflowing margins. | ||||
* | drop an unnecessary level of indent | Kartik K. Agaram | 2022-07-20 | 1 | -23/+21 |
| | |||||
* | clean up some prints | Kartik K. Agaram | 2022-07-20 | 1 | -17/+10 |
| | | | | | (Looking at prints in Text.draw, Text.compute_fragments and Text.nearest_pos_less_than) | ||||
* | move a var closer to its use | Kartik K. Agaram | 2022-07-20 | 1 | -2/+2 |
| | |||||
* | more clearly skip prints before screen top | Kartik K. Agaram | 2022-07-20 | 1 | -21/+19 |
| | |||||
* | unify two similar functions | Kartik K. Agaram | 2022-07-19 | 2 | -24/+16 |
| | | | | | | The cost is just having to tweak a few more brittle tests. I can't actually perceive any difference in how the cursor moves when I click on text. | ||||
* | keep text from overflowing right margin | Kartik K. Agaram | 2022-07-19 | 2 | -41/+48 |
| | | | | | | | | | | I've been sloppy about this so far, and outside of tests I can't find any examples where it matters, but it matters in a potential fork where I'm rendering multiple columns of text. It's unfortunate that my tests have this level of brittleness. What I'd really like to assert in many of these changed lines is that the text stays inside the margins and that more text would overflow margins. | ||||
* | delete some duplicate initialization | Kartik K. Agaram | 2022-07-19 | 1 | -4/+0 |
| | |||||
* | clean up a cross-test leakage | Kartik K. Agaram | 2022-07-19 | 1 | -9/+9 |
| | |||||
* | bugfix: couple of margin-relative computations | Kartik K. Agaram | 2022-07-17 | 2 | -3/+63 |
| | |||||
* | separate data structure for each line's cache data | Kartik K. Agaram | 2022-07-17 | 7 | -70/+211 |
| | | | | I have no idea what the performance implications of this are.. | ||||
* | keep all text cache writes inside text.lua | Kartik K. Agaram | 2022-07-17 | 2 | -9/+8 |
| | |||||
* | switch to line index in a function | Kartik K. Agaram | 2022-07-17 | 1 | -2/+3 |
| | | | | - Text.screen_line_width | ||||
* | switch to line index in a function | Kartik K. Agaram | 2022-07-17 | 1 | -16/+17 |
| | | | | - Text.compute_fragments | ||||
* | make a function oblivious to line data structure | Kartik K. Agaram | 2022-07-17 | 1 | -5/+5 |
| | | | | - Text.screen_line_index | ||||
* | switch to line index in a function | Kartik K. Agaram | 2022-07-17 | 3 | -6/+6 |
| | | | | - Text.to_pos_on_line | ||||
* | switch to line index in a function | Kartik K. Agaram | 2022-07-17 | 1 | -15/+16 |
| | | | | - Text.populate_screen_line_starting_pos | ||||
* | switch to line index in a function | Kartik K. Agaram | 2022-07-17 | 3 | -4/+5 |
| | | | | - Text.in_line | ||||
* | drop some extra args | Kartik K. Agaram | 2022-07-17 | 1 | -4/+4 |
| | |||||
* | drop some redundant args when clearing the cache | Kartik K. Agaram | 2022-07-17 | 2 | -11/+11 |
| | |||||
* | fold together two largely similar cases | Kartik K. Agaram | 2022-07-17 | 1 | -28/+23 |
| | |||||
* | drop some redundant args from Text.draw | Kartik K. Agaram | 2022-07-17 | 2 | -5/+6 |
| | |||||
* | formatting | Kartik K. Agaram | 2022-07-16 | 2 | -3/+3 |
| | |||||
* | bring back a set of constants | Kartik K. Agaram | 2022-07-16 | 4 | -90/+89 |
| | | | | | It's starting to sink in that I don't want hard-coded constants inside objects. | ||||
* | more decoupling editor tests from App | Kartik K. Agaram | 2022-07-16 | 1 | -1/+1 |
| | |||||
* | more decoupling editor tests from App | Kartik K. Agaram | 2022-07-16 | 1 | -3/+3 |
| | |||||
* | more decoupling editor tests from App | Kartik K. Agaram | 2022-07-16 | 1 | -13/+13 |
| | |||||
* | more precise name | Kartik K. Agaram | 2022-07-15 | 2 | -10/+10 |
| | |||||
* | make test initializations a little more obvious | Kartik K. Agaram | 2022-07-15 | 5 | -156/+162 |
| | |||||
* | bugfix: zoom in/out hotkeys | Kartik K. Agaram | 2022-07-13 | 1 | -0/+7 |
| | | | | Broken since commit 3b36093553 5 hours ago. | ||||
* | bugfix: resize | Kartik K. Agaram | 2022-07-13 | 2 | -2/+9 |
| | |||||
* | a more radical attempt at ignoring nil y's | Kartik K. Agaram | 2022-07-13 | 2 | -7/+5 |
| | |||||
* | no, bring back that defense | Kartik K. Agaram | 2022-07-13 | 1 | -0/+4 |
| | | | | | | | | | Scenario where I ran into it: start a stroke inside a drawing press 'o' to switch to circle mode (but any mode switch will do) The problem is that Text.textinput unconditionally blows away all .y values. Sometimes we have to wait for a draw() to fill them back in. | ||||
* | a bug I've never run into | Kartik K. Agaram | 2022-07-13 | 3 | -5/+10 |
| | | | | | | The previous commit was failing inside a test that I can't reproduce manually. Perhaps it's something about how often draw/update run in practice. Anyways, it's definitely uncovered a real issue. | ||||
* | hoist couple of variables out | Kartik K. Agaram | 2022-07-13 | 1 | -3/+6 |
| | | | | This seems to uncover a corner case I'd never run into until now. | ||||
* | use available variables | Kartik K. Agaram | 2022-07-13 | 1 | -1/+1 |
| | |||||
* | more idiomatic variable names | Kartik K. Agaram | 2022-07-13 | 1 | -5/+5 |
| | |||||
* | chunking by simple local variable | Kartik K. Agaram | 2022-07-13 | 1 | -4/+6 |
| | |||||
* | make local functions look different | Kartik K. Agaram | 2022-07-13 | 1 | -6/+6 |
| | |||||
* | more chunks, same approach | Kartik K. Agaram | 2022-07-13 | 1 | -3/+6 |
| | |||||
* | chunk up some long lines | Kartik K. Agaram | 2022-07-13 | 1 | -32/+32 |
| | | | | | Hopefully this is more approachable. Though now I'm creating two new temporary functions on every draw. Whether I need them or not. | ||||
* | . | Kartik K. Agaram | 2022-07-12 | 1 | -0/+2 |
| | |||||
* | drop final mention of state global beyond main.lua | Kartik K. Agaram | 2022-07-12 | 1 | -29/+28 |
| | | | | | | | | | | This is all unfortunate in several ways - lots of functions have extra args - table lookups where we used to have a simple variable - program initialization is a lot more delicate Let's see if it was worthwhile. Can we now actually build around the editor component in forks? | ||||
* | replace globals with args in a few functions | Kartik K. Agaram | 2022-07-12 | 4 | -78/+81 |
| | | | | | | | | | - Drawing.draw_shape - Drawing.draw_pending_shape - Drawing.in_drawing - Drawing.find_or_insert_point - Drawing.near - Drawing.pixels |