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 doesn't work while Shift key is pressed, in the script editor #49517

Closed
Chaosus opened this issue Jun 11, 2021 · 2 comments · Fixed by #49612
Closed

Backspace doesn't work while Shift key is pressed, in the script editor #49517

Chaosus opened this issue Jun 11, 2021 · 2 comments · Fixed by #49612

Comments

@Chaosus
Copy link
Member

Chaosus commented Jun 11, 2021

Godot version

fbb5a54

System information

Windows 10

Issue description

Pressing the backspace key to remove the previous symbol in the script editor does not work if the shift is pressed. This is regression not presented in 3.3.

Steps to reproduce

Create a script and try.

Minimal reproduction project

Too easy to reproduce.

@Chaosus Chaosus added this to the 4.0 milestone Jun 11, 2021
@Chaosus Chaosus changed the title Backspace doesn't work if Shift key is pressed in script editor Backspace doesn't work while Shift key is pressed, in the script editor Jun 11, 2021
@Calinou
Copy link
Member

Calinou commented Jun 11, 2021

I can confirm this on master Git dfc88bc on Linux. I was pretty sure I reported it at some point, but I can't find an issue about it right now.

@EricEzaM
Copy link
Contributor

A regression from #43663 because the shortcuts use "exact match" for their checking. This is because we have things like Ctrl+Backspace, etc which perform different actions, and we do not want overlap of those.

The safe option here I suppose is to just add Shift+Backspace to the list of commands, as done in the PR linked above.

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

Successfully merging a pull request may close this issue.

3 participants