Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | add state arg to Drawing.draw | Kartik K. Agaram | 2022-07-12 | 2 | -16/+16 |
| | |||||
* | add state arg to Text.keychord_pressed | Kartik K. Agaram | 2022-07-12 | 2 | -105/+105 |
| | |||||
* | . | Kartik K. Agaram | 2022-07-12 | 1 | -1/+1 |
| | |||||
* | add state arg to Drawing.mouse_released | Kartik K. Agaram | 2022-07-12 | 2 | -21/+21 |
| | |||||
* | add state arg to Drawing.keychord_pressed | Kartik K. Agaram | 2022-07-12 | 2 | -33/+33 |
| | |||||
* | add state arg to Drawing.mouse_pressed | Kartik K. Agaram | 2022-07-12 | 2 | -16/+16 |
| | |||||
* | add state arg to some functions | Kartik K. Agaram | 2022-07-12 | 3 | -31/+31 |
| | | | | | | - Text.draw - Text.draw_cursor - Text.draw_search_bar | ||||
* | call edit rather than App callbacks in tests | Kartik K. Agaram | 2022-07-12 | 3 | -188/+231 |
| | |||||
* | start passing in Editor_state explicitly | Kartik K. Agaram | 2022-07-12 | 4 | -231/+232 |
| | | | | | | | | | | | | 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 | 3 | -88/+4 |
| | |||||
* | group all editor globals | Kartik K. Agaram | 2022-07-12 | 11 | -1964/+1974 |
| | | | | 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 | 2 | -24/+21 |
| | |||||
* | button framework is at the app level | Kartik K. Agaram | 2022-07-12 | 2 | -3/+4 |
| | |||||
* | start decoupling editor tests from App | Kartik K. Agaram | 2022-07-11 | 2 | -100/+100 |
| | |||||
* | make colors easier to edit | Kartik K. Agaram | 2022-07-11 | 8 | -37/+38 |
| | |||||
* | experiment: new edit namespace | Kartik K. Agaram | 2022-07-11 | 3 | -416/+463 |
| | | | | | | | | 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. | ||||
* | stop pretending globals are local | Kartik K. Agaram | 2022-07-11 | 6 | -19/+5 |
| | | | | | One advantage of this approach: we don't end up with multiple lexical scopes containing duplicates of the same modules. | ||||
* | . | Kartik K. Agaram | 2022-07-11 | 1 | -1/+2 |
| | |||||
* | fix a variable name | Kartik K. Agaram | 2022-07-11 | 1 | -3/+3 |
| | |||||
* | skip multiple consecutive whitespace | Kartik K. Agaram | 2022-07-11 | 2 | -2/+49 |
| | |||||
* | support other whitespace chars in word movements | Kartik K. Agaram | 2022-07-11 | 2 | -5/+33 |
| | |||||
* | flesh out some tests for word movements | Kartik K. Agaram | 2022-07-11 | 1 | -0/+125 |
| | |||||
* | 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 | 2 | -11/+11 |
| | | | | - 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 | 3 | -8/+8 |
| | | | | - Text.pos_at_start_of_cursor_screen_line | ||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 3 | -6/+5 |
| | | | | - Text.to_pos_on_line | ||||
* | indent | Kartik K. Agaram | 2022-07-08 | 2 | -3/+12 |
| | |||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 3 | -5/+5 |
| | | | | - Text.in_line | ||||
* | add args to some functions | Kartik K. Agaram | 2022-07-08 | 3 | -15/+15 |
| | | | | | | | | | - 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 | 2 | -10/+10 |
| | | | | | | | | | | | 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 |
| | |||||
* | affirm a priority | Kartik K. Agaram | 2022-07-08 | 1 | -0/+2 |
| | |||||
* | drop an arg from a function | Kartik K. Agaram | 2022-07-08 | 3 | -4/+4 |
| | |||||
* | switch arg for a function | Kartik K. Agaram | 2022-07-08 | 2 | -16/+12 |
| | |||||
* | stop scanning from start of file on every App.draw | Kartik K. Agaram | 2022-07-07 | 1 | -40/+38 |
| | |||||
* | yet another bugfix in selection management | Kartik K. Agaram | 2022-07-07 | 2 | -20/+6 |
| | | | | | | | | | | Thanks Leonard Schütz for the report! Failing scenario: click to move cursor hit backspace First backspace wasn't being doing anything earlier. | ||||
* | just keep the cursor visible after any input events | Kartik K. Agaram | 2022-07-06 | 1 | -2/+8 |
| | | | | This is much nicer. | ||||
* | show cursor immediately after a mouse click | Kartik K. Agaram | 2022-07-06 | 1 | -0/+2 |
| | | | | | I've been subtly finding myself waiting for the cursor to stop blinking to see if I clicked on the right place. | ||||
* | better error message | Kartik K. Agaram | 2022-07-05 | 1 | -0/+3 |
| | | | | https://tildegit.org/akkartik/lines.love/issues/1 | ||||
* | make freehand drawings smoother | Kartik K. Agaram | 2022-07-05 | 1 | -1/+17 |
| | | | | | | | | | | | | Now I might actually use them more, and maybe I can start considering taking out some shapes. Do I really need circles if I don't provide ellipses? Thanks Ivan Reese for the feedback. "What drawings does your tool encourage?" Minor note: taking out the deepcopy creates a cute little string like effect, where the curve grows tighter the slower you draw it. | ||||
* | 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 | 2 | -8/+15 |
| | | | | | 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: 'escape' to cancel a stroke | Kartik K. Agaram | 2022-07-05 | 3 | -6/+28 |
| | | | | | | This has been broken since commit b544e8c357 on May 17 :/ I'm just undoing that commit, which turns out to be completely unnecessary. And adding a test. |