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

Backspace often deletes two spaces at once #660

Closed
joschol opened this issue May 17, 2023 · 6 comments
Closed

Backspace often deletes two spaces at once #660

joschol opened this issue May 17, 2023 · 6 comments
Labels
✅enhancement 🌐i18n Localization/Internationalization ❓question ui
Milestone

Comments

@joschol
Copy link

joschol commented May 17, 2023

Hitting the backspace key would often delete two spaces (and not only one as it is supposed to). To reproduce this:

  1. enter a random word
  2. hit space bar TWICE
  3. enter another random word
  4. move the caret to the beginning of the second word and hit backspace (ONCE)
  5. often, the whole gap (containing two spaces) between the words gets deleted (and not only one space)

Windows 8.1, Notepad2 (64-bit) 4.23.04 r4766 (9b680ba)

@zufuliu
Copy link
Owner

zufuliu commented May 17, 2023

It actually delete spaces up to indentation size (e.g. 4 spaces at a time) to make text aligned to indentation after deleting (though delete with Delete is one space a time).

@zufuliu
Copy link
Owner

zufuliu commented May 18, 2023

The behavior was implemented from issues #22.

@joschol
Copy link
Author

joschol commented May 18, 2023

Thanks for the quick clarification! Most of the time, I use Notepad2 for writing and structuring longer texts, the code folding feature is very helpful here, even though it's not code I'm writing. In classic word processors the backspace key is used in another way, of course. Therefore my confusion. I might just set "Indentation size" to "1" or use an older version (4.20.11) before the implementation from issues #22. Anyway, thank for your work! Great program!

@zufuliu zufuliu added ✅enhancement 🌐i18n Localization/Internationalization ui labels May 18, 2023
@zufuliu zufuliu added this to the v4.23.06 milestone May 18, 2023
@zufuliu
Copy link
Owner

zufuliu commented May 18, 2023

Then an option could be added to disable this behavior.

@zufuliu
Copy link
Owner

zufuliu commented May 18, 2023

Added option "Backspace key smart delete spaces." on Tab Settings dialog, feel free to test latest builds (e.g. from https://github.com/zufuliu/notepad2/actions/runs/5018909219).

image

@joschol
Copy link
Author

joschol commented May 19, 2023

This works great for me! Thanks a lot!

@zufuliu zufuliu closed this as completed Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅enhancement 🌐i18n Localization/Internationalization ❓question ui
Projects
None yet
Development

No branches or pull requests

2 participants