| Commit message (Collapse) | Author | Age | Files | Lines | ||
|---|---|---|---|---|---|---|
| ... | ||||||
| * | . | Kartik K. Agaram | 2022-05-29 | 1 | -6/+0 | |
| | | ||||||
| * | some helpers | Kartik K. Agaram | 2022-05-29 | 1 | -0/+16 | |
| | | | | | | | | | | | | | I spent some time trying to reduce the duplication between identical operations with and without the shift key pressed. However it makes things harder to understand. If you try to process selection in App.keychord_pressed in main.lua, you have to sometimes process the selection before (e.g. including the state of the cursor _before_ an arrow key takes effect), and sometimes after (e.g. copying the selection before resetting it, even though it doesn't include a shift key) Let's just leave things as they are. | |||||
| * | . | Kartik K. Agaram | 2022-05-29 | 1 | -4/+4 | |
| | | ||||||
| * | cut/copy selected text to clipboard | Kartik K. Agaram | 2022-05-29 | 1 | -0/+45 | |
| | | ||||||
| * | selecting text and deleting selections | Kartik K. Agaram | 2022-05-29 | 3 | -4/+310 | |
| | | | | | | | I've written a few tests for delete_selection, but the way different operations initialize the selection seems fairly standard and not worth testing so far. | |||||
| * | bugfix for non-ASCII | Kartik K. Agaram | 2022-05-29 | 1 | -3/+4 | |
| | | ||||||
| * | update some documentation | Kartik K. Agaram | 2022-05-29 | 1 | -3/+6 | |
| | | ||||||
| * | extract a couple of methods | Kartik K. Agaram | 2022-05-28 | 1 | -5/+25 | |
| | | ||||||
| * | bugfix: include shift keys in modifier_down | Kartik K. Agaram | 2022-05-28 | 1 | -3/+25 | |
| | | ||||||
| * | extract a couple more methods | Kartik K. Agaram | 2022-05-28 | 1 | -26/+34 | |
| | | ||||||
| * | move some code | Kartik K. Agaram | 2022-05-28 | 2 | -28/+28 | |
| | | | | | | I had this idea originally to keep text.lua oblivious to drawings. But that hasn't been true for some time. Losing battle. | |||||
| * | extract a few methods | Kartik K. Agaram | 2022-05-28 | 1 | -107/+121 | |
| | | ||||||
| * | move | Kartik K. Agaram | 2022-05-28 | 1 | -36/+38 | |
| | | ||||||
| * | . | Kartik K. Agaram | 2022-05-28 | 1 | -1/+1 | |
| | | ||||||
| * | scroll on backspace | Kartik K. Agaram | 2022-05-28 | 1 | -0/+68 | |
| | | ||||||
| * | . | Kartik K. Agaram | 2022-05-28 | 1 | -4/+4 | |
| | | ||||||
| * | . | Kartik K. Agaram | 2022-05-28 | 1 | -8/+8 | |
| | | ||||||
| * | fix a regression in line wrapping | Kartik K. Agaram | 2022-05-28 | 1 | -1/+44 | |
| | | ||||||
| * | scroll on enter | Kartik K. Agaram | 2022-05-28 | 1 | -0/+36 | |
| | | ||||||
| * | . | Kartik K. Agaram | 2022-05-27 | 1 | -1/+1 | |
| | | ||||||
| * | tweak modifier keys to include 'shift' | Kartik K. Agaram | 2022-05-27 | 1 | -1/+4 | |
| | | ||||||
| * | paste in text with M-v | Kartik K. Agaram | 2022-05-26 | 2 | -0/+13 | |
| | | ||||||
| * | . | Kartik K. Agaram | 2022-05-25 | 1 | -1/+0 | |
| | | ||||||
| * | few more transitions between shapes | Kartik K. Agaram | 2022-05-25 | 1 | -2/+10 | |
| | | ||||||
| * | typos | Kartik K. Agaram | 2022-05-25 | 1 | -2/+2 | |
| | | ||||||
| * | M-left/M-right for word-based motions | Kartik K. Agaram | 2022-05-25 | 2 | -0/+23 | |
| | | ||||||
| * | extract couple of functions | Kartik K. Agaram | 2022-05-25 | 1 | -34/+42 | |
| | | ||||||
| * | couple of renames | Kartik K. Agaram | 2022-05-25 | 1 | -23/+14 | |
| | | ||||||
| * | keep repeated cursor up/down motions from drifting left | Kartik K. Agaram | 2022-05-25 | 1 | -1/+5 | |
| | | ||||||
| * | assert for a bug I saw a while ago but can no longer reproduce | Kartik K. Agaram | 2022-05-25 | 1 | -0/+1 | |
| | | | | | | I saw screen_top not at start of screen line, but at cursor location in middle of line. | |||||
| * | bugfix: don't append metadata when it already exists | Kartik K. Agaram | 2022-05-25 | 1 | -2/+6 | |
| | | ||||||
| * | bugfix | Kartik K. Agaram | 2022-05-25 | 1 | -0/+41 | |
| | | | | | | I spent 20 minutes converting a manual test to a reproducible automated one, but in the process I knew exactly what the problem was. Nice. | |||||
| * | test and App helper for mouse clicks | Kartik K. Agaram | 2022-05-25 | 2 | -1/+24 | |
| | | ||||||
| * | no, make sure to compute line width after screen dimensions | Kartik K. Agaram | 2022-05-25 | 3 | -6/+8 | |
| | | ||||||
| * | couple more tests | Kartik K. Agaram | 2022-05-25 | 4 | -18/+83 | |
| | | | | | Along with the App helpers needed for them. | |||||
| * | bugfix: down arrow doesn't scroll up unnecessarily | Kartik K. Agaram | 2022-05-24 | 1 | -4/+46 | |
| | | | | | up arrow doesn't seem to have the symmetric issue. | |||||
| * | bugfix | Kartik K. Agaram | 2022-05-24 | 1 | -5/+46 | |
| | | | | | Found while reading https://www.gutenberg.org/ebooks/52091 | |||||
| * | get rid of debug variables | Kartik K. Agaram | 2022-05-23 | 2 | -16/+6 | |
| | | ||||||
| * | keep one screen line of overlap on pagedown | Kartik K. Agaram | 2022-05-23 | 2 | -19/+44 | |
| | | | | | | | | | | | | | | | | | | | | | I'm now extracting the concern of computing line.screen_line_starting_pos out of Text.draw. Earlier I had to make sure I ran through the whole line to compute screen_line_starting_pos, but that had the side-effect of updating Screen_bottom1.pos as well with lines that had never been rendered. In this process I hit my first bug due to an accidental global. It doesn't show up in the patch because I accidentally deleted a local declaration. (I thought I didn't need screen_line_starting_pos anymore, deleted everywhere, then brought it back everywhere from the bottom of the function up, but forgot to put back the very first occurrence.) The amount of yoyoing this caused between App.draw and Text.draw, I very much have spaghetti on my hands. Accidental globals are _terrible_ in a program with tests. Cross test contamination X-( | |||||
| * | a few tests for pageup, and a bugfix | Kartik K. Agaram | 2022-05-23 | 2 | -10/+107 | |
| | | | | | It wasn't screen-line aware. Now it is. | |||||
| * | clean up test progress indicators | Kartik K. Agaram | 2022-05-23 | 1 | -11/+11 | |
| | | ||||||
| * | one bug I've repeatedly run into while testing with Moby Dick | Kartik K. Agaram | 2022-05-23 | 2 | -3/+69 | |
| | | | | | https://www.hogbaysoftware.com/posts/moby-dick-workout | |||||
| * | scrolling with up arrow | Kartik K. Agaram | 2022-05-23 | 1 | -20/+19 | |
| | | ||||||
| * | disable all debug prints | Kartik K. Agaram | 2022-05-23 | 3 | -25/+25 | |
| | | ||||||
| * | quite the frustrating bugfix | Kartik K. Agaram | 2022-05-23 | 1 | -5/+7 | |
| | | ||||||
| * | snapshot | Kartik K. Agaram | 2022-05-23 | 2 | -22/+169 | |
| | | | | | Why the fuck is this so fucking hard? | |||||
| * | . | Kartik K. Agaram | 2022-05-23 | 1 | -1/+2 | |
| | | ||||||
| * | similar tests for cursor up | Kartik K. Agaram | 2022-05-23 | 1 | -1/+67 | |
| | | ||||||
| * | couple of tests for cursor down | Kartik K. Agaram | 2022-05-23 | 2 | -8/+68 | |
| | | ||||||
| * | bugfix: don't rely on Screen_bottom1 while scrolling | Kartik K. Agaram | 2022-05-23 | 3 | -12/+23 | |
| | | | | | | Setting up the test just right to test the thing I want to test was a rube goldberg machine of constants. | |||||