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

Redo not working when editing query parameters #1636

Closed
moltob opened this issue Nov 4, 2024 · 11 comments
Closed

Redo not working when editing query parameters #1636

moltob opened this issue Nov 4, 2024 · 11 comments
Labels
bug Something isn't working

Comments

@moltob
Copy link

moltob commented Nov 4, 2024

Describe the bug
When editing a request's query parameter's value, the redo hotkey (ctrl-y) is not working but instead performs an undo.

To Reproduce

  1. Open query parameters view of a request.
  2. Add a new query parameter
  3. Type some text into the value field
  4. Hit "CTRL-Z" once or twice to undo (works)
  5. Hit "CTRL-Y" a few times to redo (does not work, but still does undo)

Expected behavior
CTRL-Y performs a redo operation.

Platform:

  • OS: Windows (WSL), German keyboard layout
  • vscode version: 1.95.1
  • node version: 21.7.3
  • extension version: 2.29.6

Are you using the free version/paid version/trial:
Paid.

@moltob moltob added the bug Something isn't working label Nov 4, 2024
@rangav
Copy link
Collaborator

rangav commented Nov 4, 2024

Hi @moltob thanks for reporting the issue.

can you please confirm your subscription using below form
https://www.thunderclient.com/verify

@moltob
Copy link
Author

moltob commented Nov 5, 2024

Done

@rangav
Copy link
Collaborator

rangav commented Nov 5, 2024

Thanks for the confirmation. I wiill verify and fix it asap.

@rangav
Copy link
Collaborator

rangav commented Nov 10, 2024

redo keyboard shortcut is ctrl + Shft + Z

@moltob
Copy link
Author

moltob commented Nov 10, 2024

Is that just a comment or the supposed solution?

The "redo" shortcut in most applications is CTRL-Y. The mentioned CTRL-SHIFT-Z is an alternative sometimes found in applications (I read Photoshop). Windows therefore binds both shortcuts to "redo" automatically.

Regardless of that, my VSCode shortcuts choose to take over the system settings, meaning CTRL-Y is still an active "redo" shortcut:

image

Why would an extension choose to change that? And in fact make it even worse and bind "undo" to CTRL-Y as well?

@rangav
Copy link
Collaborator

rangav commented Nov 10, 2024

We did not change anything to keybindings

We implemented standard keybindings for undo and redo

See vscode keyboard shortcuts
Screenshot 2024-11-10 at 11 38 27

@moltob
Copy link
Author

moltob commented Nov 10, 2024

It is not up to the extension to decide on bindings:

image

@rangav
Copy link
Collaborator

rangav commented Nov 10, 2024

Did you change to keybindings for redo?

@moltob
Copy link
Author

moltob commented Nov 10, 2024

No, as I wrote above, this is Windows default.

@rangav
Copy link
Collaborator

rangav commented Nov 10, 2024

Thanks for the clarification, I will look into the issue.

@rangav
Copy link
Collaborator

rangav commented Dec 8, 2024

This bug is fixed in v2.32.9, Please test it and let me know your feedback.

https://github.com/thunderclient/thunder-client-support/releases/tag/v2.32.9

@rangav rangav closed this as completed Dec 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants