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

Menu item for copying only "Message" column #302

Closed
arkinform opened this issue Sep 20, 2017 · 17 comments
Closed

Menu item for copying only "Message" column #302

arkinform opened this issue Sep 20, 2017 · 17 comments

Comments

@arkinform
Copy link

In our project we use DebugOutput for logging SQL quiries with parameter values. And we often need to copy some querу and execute it in other program. But DebugViewPP always copy with all column values. Will be very usefull to make separate context menu item for copy selected text only from "Message" column.

@janwilmans
Copy link
Member

You can already select a piece of the message to copy it, are you aware of that? and still think its useful of have a separate 'copy messsage' item ?

@arkinform
Copy link
Author

  1. I can not select "piece of the message" in multiple rows
  2. Selecting long text with mouse is not very convenient

@janwilmans
Copy link
Member

I see, I'll create something soon!

@janwilmans janwilmans self-assigned this Sep 30, 2017
@arkinform
Copy link
Author

I see, I'll create something soon!

Do you plan to make something like "Copy message" in near future?

@janwilmans
Copy link
Member

I totally forgot about this one, sorry about that, I'm in the middle of a small unicode refactoring right now, but I will see what I can do.

@janwilmans
Copy link
Member

I've added a context-menu item, please try it:
https://github.com/CobaltFusion/DebugViewPP/releases/tag/v1.8.0.25

@janwilmans
Copy link
Member

implemented in 4250e58

@arkinform
Copy link
Author

"Copy messages" item works fine, thank you.

A few remarks:

  1. I think that it will be better to preserve old "Copy" item before "Copy messages" item in menu for backward compatibility.
  2. Please, add shortcut for "Copy messages", for example, Shift + Ctrl + C
  3. Selection with Shift + Up/Down does not work

@janwilmans
Copy link
Member

rework.

@janwilmans janwilmans reopened this Aug 21, 2018
@janwilmans
Copy link
Member

Selection with Shift + Up/Down should be fixed in
https://github.com/CobaltFusion/DebugViewPP/releases/tag/v1.8.0.27

@arkinform
Copy link
Author

  1. I have checked selection with Shift + Up/Down, works fine
  2. When I click right mouse button on "Process" column then I see old "Copy" item and it does not work

@janwilmans
Copy link
Member

What do you mean with 'does not work' ?
you expected it to copy the process name?
I tried it and it copies the whole line (like the old copy) here...

@arkinform
Copy link
Author

I tried it and it copies the whole line (like the old copy) here...

In my case keyboard shortcut Ctrl+C works fine but mouse click on menu item "Copy" (visible only for "Process" column) does not copy anything.

And I think that it is oddly to see "Copy" menu item only for "Process" column.

@arkinform
Copy link
Author

image

@janwilmans
Copy link
Member

The fact that copy on one context menu works, but not on the other is very strange, the logic behind the two menu items is identical:

image

@arkinform
Copy link
Author

Sorry for delay. I have tried last build v1.8.0.34, in this build everything works fine.
Shortcut Shift + Ctrl + C works fine too but I don't see shortcut hint for item "Copy Messages" in context menu.

@janwilmans
Copy link
Member

I've added the shortcut hint, it well be in the next release. Thanks for testing, it is much appreciated!

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

No branches or pull requests

2 participants