Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | simplify cursor-on-screen check | Kartik K. Agaram | 2022-08-17 | 1 | -6/+6 | |
| | ||||||
* | swap return values | Kartik K. Agaram | 2022-08-17 | 1 | -5/+5 | |
| | ||||||
* | obsolete comment | Kartik K. Agaram | 2022-08-16 | 1 | -2/+0 | |
| | ||||||
* | move | Kartik K. Agaram | 2022-08-15 | 1 | -28/+28 | |
| | ||||||
* | drop some unnecessary calls | Kartik K. Agaram | 2022-08-15 | 1 | -6/+0 | |
| | ||||||
* | stop confusingly reading a global | Kartik K. Agaram | 2022-08-15 | 1 | -2/+2 | |
| | | | | | | | The way Text.draw is called by edit.draw, we know it'll never be called for lines above screen_top1.line. Comparing every line on screen with screen_top1 makes no sense. The intent is really just to compare with screen_top1 only for the first line, and otherwise to ignore this check. | |||||
* | bugfix: handle drawings when updating screen top | Kartik K. Agaram | 2022-08-11 | 1 | -0/+1 | |
| | ||||||
* | rename | Kartik K. Agaram | 2022-08-11 | 1 | -24/+24 | |
| | ||||||
* | bugfix: pagedown was sometimes bouncing up | Kartik K. Agaram | 2022-08-10 | 1 | -1/+1 | |
| | ||||||
* | bugfix: backspace from start of final line | Kartik K. Agaram | 2022-08-10 | 1 | -1/+5 | |
| | ||||||
* | unnecessary args | Kartik K. Agaram | 2022-08-10 | 1 | -1/+1 | |
| | ||||||
* | click to the left of a line | Kartik K. Agaram | 2022-07-29 | 1 | -2/+1 | |
| | ||||||
* | bugfix: search | Kartik K. Agaram | 2022-07-21 | 1 | -1/+1 | |
| | | | | | Broken since commit 188bbc73 9 days ago :/ At least we have a test for it now. | |||||
* | use line cache for drawings as well | Kartik K. Agaram | 2022-07-20 | 1 | -26/+26 | |
| | ||||||
* | bugfix: where cursor is drawn | Kartik K. Agaram | 2022-07-20 | 1 | -4/+11 | |
| | | | | | | | 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. | |||||
* | . | Kartik K. Agaram | 2022-07-20 | 1 | -0/+2 | |
| | ||||||
* | exclude left margin from my word-split heuristic | Kartik K. Agaram | 2022-07-20 | 1 | -1/+1 | |
| | | | | | 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 | 1 | -5/+2 | |
| | | | | 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 | 1 | -11/+4 | |
| | | | | | | 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 | 1 | -5/+15 | |
| | | | | | | | | | | 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. | |||||
* | bugfix: couple of margin-relative computations | Kartik K. Agaram | 2022-07-17 | 1 | -2/+2 | |
| | ||||||
* | separate data structure for each line's cache data | Kartik K. Agaram | 2022-07-17 | 1 | -50/+58 | |
| | | | | 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 | 1 | -5/+5 | |
| | ||||||
* | 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 | 1 | -3/+3 | |
| | | | | - 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 | 1 | -1/+2 | |
| | | | | - Text.in_line | |||||
* | drop some redundant args when clearing the cache | Kartik K. Agaram | 2022-07-17 | 1 | -10/+10 | |
| | ||||||
* | drop some redundant args from Text.draw | Kartik K. Agaram | 2022-07-17 | 1 | -2/+3 | |
| | ||||||
* | bring back a set of constants | Kartik K. Agaram | 2022-07-16 | 1 | -3/+3 | |
| | | | | | It's starting to sink in that I don't want hard-coded constants inside objects. | |||||
* | replace globals with args in a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -3/+3 | |
| | | | | | | | | | - Drawing.draw_shape - Drawing.draw_pending_shape - Drawing.in_drawing - Drawing.find_or_insert_point - Drawing.near - Drawing.pixels | |||||
* | deduce left/right from state where possible | Kartik K. Agaram | 2022-07-12 | 1 | -64/+64 | |
| | ||||||
* | left/right margin -> left/right coordinates | Kartik K. Agaram | 2022-07-12 | 1 | -31/+31 | |
| | | | | | Editor state initialization now depends on window dimensions, so we have to more carefully orchestrate startup. | |||||
* | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -107/+93 | |
| | | | | | | | | | | | | | - Text.cursor_at_final_screen_line - Text.move_cursor_down_to_next_text_line_while_scrolling_again_if_necessary - Text.snap_cursor_to_bottom_of_screen - Text.in_line - Text.to_pos_on_line - Text.to2 - Text.to1 - Text.previous_screen_line - Text.tweak_screen_top_and_cursor - Text.redraw_all | |||||
* | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -17/+17 | |
| | | | | | - Text.pos_at_start_of_cursor_screen_line - Text.cursor_past_screen_bottom | |||||
* | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -26/+26 | |
| | | | | | | - Text.left - Text.right - Text.right_without_scroll | |||||
* | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -12/+12 | |
| | | | | | | | - record_undo_event - undo_event - redo_event - snapshot | |||||
* | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -4/+4 | |
| | | | | | | | | | | | - Text.draw_highlight - Text.clip_selection - Text.selection - Text.cut_selection - Text.delete_selection - Text.delete_selection_without_undo - Text.mouse_pos - Text.to_pos | |||||
* | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -122/+122 | |
| | | | | | | | | | | | | | - Text.insert_at_cursor - Text.insert_return - Text.pageup - Text.pagedown - Text.up - Text.down - Text.start_of_line - Text.end_of_line - Text.word_left - Text.word_right | |||||
* | add state arg to Text.textinput | Kartik K. Agaram | 2022-07-12 | 1 | -8/+8 | |
| | ||||||
* | add state arg to schedule_save | Kartik K. Agaram | 2022-07-12 | 1 | -6/+6 | |
| | ||||||
* | add state arg to Text.keychord_pressed | Kartik K. Agaram | 2022-07-12 | 1 | -104/+104 | |
| | ||||||
* | add state arg to some functions | Kartik K. Agaram | 2022-07-12 | 1 | -22/+22 | |
| | | | | | | - Text.draw - Text.draw_cursor - Text.draw_search_bar | |||||
* | group all editor globals | Kartik K. Agaram | 2022-07-12 | 1 | -299/+299 | |
| | | | | We're still accessing them through a global. But we'll change that next. |