madskristensen / RestClientVS

Other
66 stars 14 forks source link

Not sure if this is a VS bug really... #21

Open SQL-MisterMagoo opened 2 years ago

SQL-MisterMagoo commented 2 years ago

Describe the bug If the .http file is part of a solution then you cannot copy from the ResponseControl TextBox using the keyboard Ctrl+C shortcut.

To Reproduce Steps to reproduce the behavior:

  1. Open any Solution in VS
  2. Use File...Open to open an .http file
  3. Run one of the REST calls in the .http file
  4. Select some text in the Results text area and use the keyboard shortcut to copy the text
  5. Paste into the text editor pane - it will paste the text you copied.
  6. Now (you can leave the .http file open) add the .http to the Solution using Add..Existing Item
  7. Select some different text in the results pane and use the keyboard shortcut to copy the text
  8. Paste into the text editor pane - it will not paste anything - nothing was copied to the clipboard.

Expected behavior Using keyboard shortcuts should work whether a file is part of a Solution or not.

Screenshots When the file is not part of the solution, keyboard shortcuts work image

When the file is part of the solution, keyboard shortcuts do not work image

Additional context I logged this here so it can be tracked, but I suspect it must be a VS issue, so I will log it there as well and cross-reference.

SQL-MisterMagoo commented 2 years ago

Logged for the VS team here

SQL-MisterMagoo commented 2 years ago

The VS team have closed the log with this comment: "This is an issue in the extension, please report it in https://github.com/madskristensen/RestClientVS/issues. Most likely command routing is not setup correctly and somehow is being affected if file is part of a project."

If anyone has any idea what they mean by this could you comment here and I am happy to investigate further.

madskristensen commented 2 years ago

I thought that's what you fixed in https://github.com/madskristensen/RestClientVS/pull/19, was it not?

SQL-MisterMagoo commented 2 years ago

Th fix in #19 enabled the Ctrl-C to work, but it only works if the file is not part of a solution.

I don't know what "Command Routing" is being referred to by the VS support - it sounds like they are thinking of VS Extensibility commands, which this is not... and I don't know why it matters if the file is part of a solution.