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

Ctrl+C: Copy Screenshot to Clipboard #342

Closed
sriediger opened this issue Dec 2, 2021 · 8 comments
Closed

Ctrl+C: Copy Screenshot to Clipboard #342

sriediger opened this issue Dec 2, 2021 · 8 comments
Labels

Comments

@sriediger
Copy link

Currently Ctrl+C copies a screenshot of the graph to the clipboard.
This prevents that a user can use Ctrl+C to copy text from an edit control of a property page to the clipboard.
My biggest wish would be to assign a different key combination to "Copy Screenshot to Clipboard" and avoid standard Windows shortcuts.

Thank you very much in advance.

P.S: GraphStudioNext is great :-)

@roman380
Copy link
Collaborator

roman380 commented Dec 2, 2021

Do you have any suggestions, that make sense and don't collide with anything else?

@sriediger
Copy link
Author

A difficult question.

  • Perhaps Ctrl+Shift+Print Screen, if this is possible.
  • This combination is not so often used in programs, compared to Ctrl+C
  • Print Screen is mentally associated with some kind of screenshot
  • I know that Ctrl+Shift+Print Screen would trigger a screenshot of the whole desktop, but only Print Screen alone does the same.

A customization would be a too big effort for the problem, right?

@roman380
Copy link
Collaborator

roman380 commented Dec 2, 2021

I mean I tihnk your concern is totally valid and it's a good idea to change, the question is whether you have a good candidate in mind or somoene is yet to figure that out. PrintScreen itself is handled by OS for desktop capture and also Alt+PrintScreen IIRC. Maybe it should rather be a traditional letter based hotkey, just another one without known conflicts.

@clsid2
Copy link

clsid2 commented Dec 2, 2021

Ctrl+F12 is currently used to save to image file. Maybe use Ctrl+Shift+F12 for clipboard? And add it to the menu as well.

@roman380
Copy link
Collaborator

roman380 commented Dec 3, 2021

I will leave #343 for guys to hav a look or otherwise merge in a while (a minor change and makes sense).

And add it to the menu as well.

They are currently both in menu, one under File, and the other under View. Maybe a bit messy but not that bad.

@cplussharp
Copy link
Owner

I don't think we should just change the short-cut. That could confuse existing users! The more important question is, why is the clipboard command executed on the graph? Does it just bubble up from the the property page or is it never executed on the property page in the first place?

@roman380
Copy link
Collaborator

roman380 commented Dec 5, 2021

These might be two related issues, better shortcut & also disablement of scope where accelerators apply. The original request overall looks valid. Repo steps:

  • add a filter like AC3Filter, for example, into graph and open its properties, switch to page with edit control there
  • Ctrl+C takes us to CGraphView::OnViewCopyScreenshotToClipboard

@cplussharp cplussharp added the bug label Dec 5, 2021
@cplussharp
Copy link
Owner

@roman380 thanks for your change, I just picked the last commit ( 4dacf1c ) to fix the problem. So the Ctrl+C is not handled in the main window anymore, when a property page (or any other child window/dialog) is active.

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

No branches or pull requests

4 participants