| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | line.y -> line_cache.starty in a few more places | Kartik K. Agaram | 2022-07-27 | 1 | -8/+8 |
| | | | | | | | Disquieting that none of my tests caught these. On the other hand, I also haven't noticed any issues in practice. Perhaps cache invalidation is often unnecessary. | ||||
| * | have file API operate on state object | Kartik K. Agaram | 2022-07-25 | 1 | -2/+2 |
| | | |||||
| * | extract method | Kartik K. Agaram | 2022-07-23 | 1 | -0/+11 |
| | | |||||
| * | move drawing.starty into line cache | Kartik K. Agaram | 2022-07-20 | 1 | -4/+4 |
| | | |||||
| * | use line cache for drawings as well | Kartik K. Agaram | 2022-07-20 | 1 | -4/+4 |
| | | |||||
| * | separate data structure for each line's cache data | Kartik K. Agaram | 2022-07-17 | 1 | -8/+13 |
| | | | | | 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 | -4/+3 |
| | | |||||
| * | switch to line index in a function | Kartik K. Agaram | 2022-07-17 | 1 | -2/+2 |
| | | | | | - Text.to_pos_on_line | ||||
| * | switch to line index in a function | Kartik K. Agaram | 2022-07-17 | 1 | -2/+2 |
| | | | | | - Text.in_line | ||||
| * | drop some extra args | Kartik K. Agaram | 2022-07-17 | 1 | -4/+4 |
| | | |||||
| * | 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 | 1 | -3/+3 |
| | | |||||
| * | formatting | Kartik K. Agaram | 2022-07-16 | 1 | -1/+1 |
| | | |||||
| * | bring back a set of constants | Kartik K. Agaram | 2022-07-16 | 1 | -7/+6 |
| | | | | | | It's starting to sink in that I don't want hard-coded constants inside objects. | ||||
| * | more precise name | Kartik K. Agaram | 2022-07-15 | 1 | -3/+10 |
| | | |||||
| * | make test initializations a little more obvious | Kartik K. Agaram | 2022-07-15 | 1 | -7/+17 |
| | | |||||
| * | a bug I've never run into | Kartik K. Agaram | 2022-07-13 | 1 | -2/+8 |
| | | | | | | | 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. | ||||
| * | replace globals with args in a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -2/+2 |
| | | | | | | | | | | - Drawing.draw_shape - Drawing.draw_pending_shape - Drawing.in_drawing - Drawing.find_or_insert_point - Drawing.near - Drawing.pixels | ||||
| * | correct a mis-named threshold | Kartik K. Agaram | 2022-07-12 | 1 | -0/+2 |
| | | |||||
| * | deduce left/right from state where possible | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | left/right margin -> left/right coordinates | Kartik K. Agaram | 2022-07-12 | 1 | -16/+19 |
| | | | | | | 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 | -1/+1 |
| | | | | | | | | - Drawing.current_drawing - Drawing.select_shape_at_mouse - Drawing.select_point_at_mouse - Drawing.select_drawing_at_mouse | ||||
| * | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -10/+10 |
| | | | | | | | | | | | | | | - 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 | -2/+2 |
| | | | | | | - 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 | -3/+3 |
| | | | | | | - Text.search_next - Text.search_previous | ||||
| * | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -15/+15 |
| | | | | | | | | - record_undo_event - undo_event - redo_event - snapshot | ||||
| * | add state arg to a few functions | Kartik K. Agaram | 2022-07-12 | 1 | -3/+3 |
| | | | | | | | | | | | | - 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 | -2/+2 |
| | | | | | | | | | | | | | | - 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 | -1/+1 |
| | | |||||
| * | add state arg to schedule_save | Kartik K. Agaram | 2022-07-12 | 1 | -12/+12 |
| | | |||||
| * | add state arg to Drawing.update | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | add state arg to Drawing.draw | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | add state arg to Text.keychord_pressed | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | . | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | add state arg to Drawing.mouse_released | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | add state arg to Drawing.keychord_pressed | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | add state arg to Drawing.mouse_pressed | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | add state arg to some functions | Kartik K. Agaram | 2022-07-12 | 1 | -3/+3 |
| | | | | | | | - Text.draw - Text.draw_cursor - Text.draw_search_bar | ||||
| * | call edit rather than App callbacks in tests | Kartik K. Agaram | 2022-07-12 | 1 | -0/+43 |
| | | |||||
| * | start passing in Editor_state explicitly | Kartik K. Agaram | 2022-07-12 | 1 | -123/+123 |
| | | | | | | | | | | | | | In this commit, top-level edit functions: - edit.draw - edit.update - edit.quit - edit.mouse_pressed - edit.mouse_released - edit.textinput - edit.keychord_pressed - edit.key_released | ||||
| * | initialize contains test state | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | | |||||
| * | group all editor globals | Kartik K. Agaram | 2022-07-12 | 1 | -188/+198 |
| | | | | | We're still accessing them through a global. But we'll change that next. | ||||
| * | bring couple more globals back to the app level | Kartik K. Agaram | 2022-07-12 | 1 | -23/+0 |
| | | |||||
| * | button framework is at the app level | Kartik K. Agaram | 2022-07-12 | 1 | -3/+0 |
| | | |||||
| * | make colors easier to edit | Kartik K. Agaram | 2022-07-11 | 1 | -1/+13 |
| | | |||||
| * | experiment: new edit namespace | Kartik K. Agaram | 2022-07-11 | 1 | -0/+446 |
| Still lots to do, but the eventual hope is that this will make this project's code easier to reuse from other LÖVE projects. One gotcha: even as we start putting code more aggressively into nested tables, tests must remain at the top-level. Otherwise they won't run. | |||||