copy/paste without escape characters doesn't seem to work anymore

Question

copy/paste without escape characters doesn't seem to work anymore

martinlippert opened this issue 3 years ago · comments

Looks like the standard copy feature to quickly pull text from the console view excluding the escape sequences (see #19) doesn't work anymore (as someone reported here: spring-projects/sts4#667)

Mihai Nita commented 3 years ago

Released

Mihai Nita · Answer 1 · Thu Aug 12 2021 06:51:51 GMT+0800 (China Standard Time)

Tested, confirmed (at least on macOS)

Mihai Nita · Answer 2 · Sat Aug 14 2021 18:34:03 GMT+0800 (China Standard Time)

Thanks Martin.

Meantime I've confirmed it on Windows and Linux.
I have a fix, committed.
Will do a release soon.

Martin Lippert · Answer 3 · Mon Aug 16 2021 20:23:21 GMT+0800 (China Standard Time)

Awesome, thanks for the quick turnaround, much appreciated.

Martin Lippert · Answer 4 · Mon Sep 06 2021 19:19:08 GMT+0800 (China Standard Time)

Any updates on this? Is there already a release available that includes this?

elab · Answer 5 · Tue Sep 28 2021 19:48:06 GMT+0800 (China Standard Time)

Thank you, Mihai for the awesome plugin and Martin for fixing. Any good news on release date?

elab · Answer 6 · Sun Oct 10 2021 20:42:43 GMT+0800 (China Standard Time)

As a temporal quick-and-dirty solution until the new version of the plugin is released, I replaced the file AnsiConsolePageParticipant.class in the net.mihai-nita.ansicon.plugin_1.4.4.202102040736.jar inside Eclipse with the new version from the main branch (which contains the fix). Copying without escape sequences does work now.

Tested with Eclipse 2021-06 and 2021-09. Compiled with AdoptOpenJDK-11.0.11+9.
Perhaps can be useful for someone.

Just replace the two jar files inside Eclipse plugins folder with the attached ones: net.mihai-nita.ansicon.plugin_1.4.4.202102040736.patched.zip (extract them from zip since GitHub doesn't support uploading of jar files). Only one file (AnsiConsolePageParticipant) has been replaced in the jars, all others are untouched. The source-jar is just for the sake of completeness.

elab · Answer 7 · Tue Oct 12 2021 18:13:50 GMT+0800 (China Standard Time)

Hmm... Strange behavior with this patch. Having some output in the Ansi Console and clicking Ctrl+C on a selected text in any other Eclipse view (e.g. code editor, commit details in history) will copy the whole content of the console (or whatever is selected there) into clipboard.

Copying via "Copy" entry from the context menu (as opposed to Ctrl+C shortcut) works properly in other views, but copies with escape characters from Ansi Console.

elab · Answer 8 · Wed Oct 13 2021 00:33:23 GMT+0800 (China Standard Time)

My next try is to define a CopyWithoutEscapesHandler (analogous to existing CopyWithEscapesHandler but calling AnsiClipboardUtils.textToClipboard() with true parameter to remove escapes) and bind it to the button "Copy text without escapes" on the console toolbar. Not as convenient as using keyboard shortcuts, but at least I can now copy text from console without escapes.

A proper solution from my pov would be intercepting copying operation in the Console (and only there) if the AnsiConsole plugin is activated.

elab · Answer 9 · Sun Sep 25 2022 16:24:27 GMT+0800 (China Standard Time)

Unfortunately, the copying without escape characters (color codes) is not working in Eclipse 2022-09 (where ANSI Console is a default part of Eclipse). In the plugin v1.4.5 and Eclipse 2022-06, it was nicely working. How can we fix that?

Martin Lippert · Answer 10 · Mon Sep 26 2022 15:09:36 GMT+0800 (China Standard Time)

@elab I would recommend to raise this as an enhancement request or bug against https://github.com/eclipse-platform/eclipse.platform.debug

Mihai Nita · Answer 11 · Wed Oct 05 2022 17:37:58 GMT+0800 (China Standard Time)

Explaining why this does not work anymore, and the fact that it was intentional, not an accident

When I've integrated this to Eclipse they discovered that some of the things I was doing were not quite OK.
For example I was intercepting the standard copy to do a "copy without escapes"

They argued (fair enough) that people who install this plugin are people interested in it, and care about its features.
They are free to uninstall it if they don't like it.
And it is clear who's at fault :-)

But doing something like that in the standard Eclipse, for everybody, goes against the Eclipse UX guidelines.
(same complaints about other things I was doing, like adding icons on the console toolbar).

All valid, I am not complaining :-)

How to fix it

You can copy using Ctrl+Insert instead of Ctrl+C (Command instead of Ctrl on MacOS)

That hotkey is in fact very old Copy hotkey (still works as Copy in standard OS widgets).

If you want a different hotkey (Ctrl+Shift+C, or whatever) you can configure it

The Enable/Disable and Copy (with/without escapes) are standard Eclipse commands and you can assign whatever hotkeys yoo want.
Preferences -- General -- Keys and search for ANSI.

File a bug / feature request as Martin suggested.

Mihai Nita · Answer 12 · Wed Oct 05 2022 19:12:00 GMT+0800 (China Standard Time)

For the default copy behavior discussion during the integration see eclipse-platform/eclipse.platform.debug#57

Reading it again, it was not so much about strictly "don't hijack the standard copy", but mostly a mixture of technical limitations / my lack of knowledge.
The context menu "Copy" in the console didn't show the "Ctrl+C" binding anymore.
And I didn't know how to really replace that command, instead of just "steal the hotkey"

So I don't want to blame it on others...

I expect there will be some fine-tunning for the next few Eclipse versions...
Feedback from users (like you :-) would greatly help with the direction of these changes.