search

CobaltFusion / DebugViewPP

DebugView++, collects, views, filters your application logs, and highlights information that is important to you!
Boost Software License 1.0
1.02k stars 149 forks source link

Menu item for copying only "Message" column #302

Closed arkinform closed 6 years ago

arkinform commented 7 years ago

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 commented 7 years ago

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 commented 7 years ago
  1. I can not select "piece of the message" in multiple rows
  2. Selecting long text with mouse is not very convenient
janwilmans commented 7 years ago

I see, I'll create something soon!

arkinform commented 6 years ago

I see, I'll create something soon!

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

janwilmans commented 6 years ago

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 commented 6 years ago

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

janwilmans commented 6 years ago

implemented in 4250e58609172f0a49d5a46d42ef31cdbc7e883b

arkinform commented 6 years ago

"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 commented 6 years ago

rework.

janwilmans commented 6 years ago

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

arkinform commented 6 years ago
  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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

image

janwilmans commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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