Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | 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. | |||||
* | make colors easier to edit | Kartik K. Agaram | 2022-07-11 | 1 | -4/+4 | |
| | ||||||
* | stop pretending globals are local | Kartik K. Agaram | 2022-07-11 | 1 | -4/+0 | |
| | | | | | One advantage of this approach: we don't end up with multiple lexical scopes containing duplicates of the same modules. | |||||
* | skip multiple consecutive whitespace | Kartik K. Agaram | 2022-07-11 | 1 | -2/+27 | |
| | ||||||
* | support other whitespace chars in word movements | Kartik K. Agaram | 2022-07-11 | 1 | -5/+11 | |
| | ||||||
* | done passing left/right margins everywhere | Kartik K. Agaram | 2022-07-08 | 1 | -16/+16 | |
| | | | | | | | The final step is to add them to: - Text.compute_fragments Doing it carefully caught 5 call-sites that my tests didn't catch. | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -14/+14 | |
| | | | | - Text.populate_screen_line_starting_pos | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -6/+6 | |
| | | | | - Text.to2 | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -5/+5 | |
| | | | | - Text.previous_screen_line | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -10/+10 | |
| | | | | - Text.snap_cursor_to_bottom_of_screen | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -10/+12 | |
| | | | | - Text.nearest_cursor_pos | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -3/+3 | |
| | | | | - Text.move_cursor_down_to_next_text_line_while_scrolling_again_if_necessary | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -2/+2 | |
| | | | | - Text.cursor_at_final_screen_line | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -5/+5 | |
| | | | | - Text.pos_at_start_of_cursor_screen_line | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -3/+2 | |
| | | | | - Text.to_pos_on_line | |||||
* | indent | Kartik K. Agaram | 2022-07-08 | 1 | -1/+4 | |
| | ||||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -2/+2 | |
| | | | | - Text.in_line | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -3/+3 | |
| | | | | | | | | | - Text.clip_selection - Text.cut_selection - Text.delete_selection - Text.delete_selection_without_undo - Text.mouse_pos - Text.to_pos | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 1 | -28/+28 | |
| | | | | | | | | | | | | - Text.pageup - Text.pagedown - Text.up - Text.down - Text.end_of_line - Text.word_left - Text.word_right - Text.left - Text.right | |||||
* | start passing left/right margins everywhere | Kartik K. Agaram | 2022-07-08 | 1 | -8/+8 | |
| | | | | | | | | | | | I have a set of changes that passes all tests, but I'm going to commit them very carefully to ensure I don't miss any call-sites. In this commit I'm adding the args to: - Text.draw - Text.tweak_screen_top_and_cursor But calls within them don't yet pass them where they should. In this manner I'm going to progress systematically from the top down. | |||||
* | rename a couple of variables | Kartik K. Agaram | 2022-07-08 | 1 | -10/+10 | |
| | ||||||
* | drop an arg from a function | Kartik K. Agaram | 2022-07-08 | 1 | -1/+1 | |
| | ||||||
* | switch arg for a function | Kartik K. Agaram | 2022-07-08 | 1 | -15/+11 | |
| | ||||||
* | optimization: moving cursor to next word | Kartik K. Agaram | 2022-07-05 | 1 | -4/+12 | |
| |