Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move rename cursor to start before the extention #565

Merged
merged 2 commits into from
Jan 25, 2025

Conversation

booth-w
Copy link
Contributor

@booth-w booth-w commented Jan 24, 2025

When renaming a file, the extention often stays the same. This change moves the cursor to the end of the name but before the extention.

(Underscore denotes cursor starting position)

test_
test_.txt
.test_
.test_.txt
te.st_.txt

Cursor position remains unaffected if the file is a directory

Copy link

netlify bot commented Jan 24, 2025

Deploy Preview for superfile canceled.

Name Link
🔨 Latest commit be237e5
🔍 Latest deploy log https://app.netlify.com/sites/superfile/deploys/6793d0b2aa909e0008b95508

Copy link
Collaborator

@lazysegtree lazysegtree left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Tested it locally and it works perfectly. Thanks @booth-w

@lazysegtree lazysegtree requested a review from yorukot January 24, 2025 19:08
@yorukot yorukot merged commit a7d1501 into yorukot:main Jan 25, 2025
4 checks passed
@yorukot
Copy link
Owner

yorukot commented Jan 25, 2025

Pretty great update thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants