fix: add filename to existing text

Open mhickman opened this issue 1 year ago • 1 comments

Instead of clobbering the input text area just append the file path at the end of the input.

This could be more clever by tracking where the cursor was and inserting the text there, but this seems better than just deleting the text.

Kapture 2025-02-13 at 15 11 45

Fixes #1179

Feb 13 '25 23:02 mhickman

Changed to track the cursor, here's how it looks now:

Kapture 2025-02-13 at 16 06 07

Feb 14 '25 00:02 mhickman

@mhickman Sorry we missed this earlier, but this is great! Just merged something similar in, but the cursor functionality here is a great addition. Would you mind rebasing and I can drop in a review afterwards? (we've had some other changes to UI since you raised this PR)

Mar 25 '25 21:03 zakiali

Hey there! We're doing some cleanup and I believe this is mostly addressed from other UX improvements, but please do feel free to reopen this PR with a rebase if it is still relevant!

Jun 16 '25 18:06 baxen