Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | 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. | |||||
* | what should happen to selection while drawing? | Kartik K. Agaram | 2022-07-05 | 1 | -0/+1 | |
| | | | | First idea: just leave it alone. Let's see how this works. | |||||
* | keep online help inside of drawing | Kartik K. Agaram | 2022-07-04 | 1 | -2/+2 | |
| | ||||||
* | bugfix | Kartik K. Agaram | 2022-07-03 | 1 | -1/+1 | |
| | | | | Gracefully degrade if previously edited file shrinks. |