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

Use escape key to exit editing #32

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Piturnah
Copy link

@Piturnah Piturnah commented Nov 24, 2022

The escape key can now be used to exit edit mode. In writing a todo, this will also cancel the new todo.

@Piturnah Piturnah marked this pull request as draft November 24, 2022 21:28
@Piturnah
Copy link
Author

Marking as draft since I just noticed a critical bug, if you are renaming a todo and escape then it will delete it. Don't know how I didn't think of that.

The escape key can now be used to exit edit mode. In writing a todo,
this will also cancel the new todo.
@Piturnah
Copy link
Author

Marking as draft since I just noticed a critical bug, if you are renaming a todo and escape then it will delete it. Don't know how I didn't think of that.

Fixed!

@Piturnah Piturnah marked this pull request as ready for review November 24, 2022 21:50
@slinlee
Copy link

slinlee commented Nov 25, 2022

@Piturnah I like the enhancement. When I try editing and pressing esc it has the same effect as pressing enter and saves the edited text.

@Piturnah
Copy link
Author

Yeah, that is the functionality I intended - but I do agree that it may feel more intuitive to have it cancel the edit you made. If the maintainer agrees then I'd be happy to give a go implementing this.

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.

2 participants