thunderclient / thunder-client-support

Thunder Client is a lightweight Rest API Client Extension for VS Code.
https://www.thunderclient.com
Other
3.64k stars 132 forks source link

Redo not working when editing query parameters #1636

Open moltob opened 3 weeks ago

moltob commented 3 weeks ago

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:

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

rangav commented 3 weeks ago

Hi @moltob thanks for reporting the issue.

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

moltob commented 2 weeks ago

Done

rangav commented 2 weeks ago

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

rangav commented 2 weeks ago

redo keyboard shortcut is ctrl + Shft + Z

moltob commented 2 weeks ago

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 commented 2 weeks ago

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 commented 2 weeks ago

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

image

rangav commented 2 weeks ago

Did you change to keybindings for redo?

moltob commented 2 weeks ago

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

rangav commented 2 weeks ago

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