Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | 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 | |
| | ||||||
* | bugfix: avoid scrolling on 'end' | Kartik K. Agaram | 2022-07-05 | 1 | -6/+14 | |
| | | | | | There's multiple ways to do this, only one of them is right, and I keep forgetting what it is. Turn it into a method. | |||||
* | bugfix: manage screen_top and cursor when resizing | Kartik K. Agaram | 2022-07-01 | 1 | -0/+35 | |
| | ||||||
* | right margin | Kartik K. Agaram | 2022-06-30 | 1 | -14/+14 | |
| | ||||||
* | rip out notion of Line_width | Kartik K. Agaram | 2022-06-30 | 1 | -21/+20 | |
| |