gui: Add Copy command feature on right-click to GUI History panel

[aniket2405]

Description:

Provide Copy command feature on right-click for commands in the GUI History panel.

Motivation and context:

For reporting bugs it's helpful to copy particular commands from history. Presently, there is just the Remove command.

How has this been tested?

Compiled and executed the Grass application locally. Can observe the new Copy command in the History Panel on right-clicking a command.

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing
  functionality to not work as before)

Checklist

• PR title provides summary of the changes and starts with one of the
  pre-defined prefixes
• My code follows the code style
  of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.

Fixes

#4385

[echoix]

@petrasovaa is there a way that exists to paste back and use a command like that? Does a json representation bring value? (In QGIS you can copy and paste the command run to populate the box again, using a json format).

[petrasovaa]

@petrasovaa is there a way that exists to paste back and use a command like that? Does a json representation bring value? (In QGIS you can copy and paste the command run to populate the box again, using a json format).

Not sure I understand, something like the --ui ?

[echoix]

@petrasovaa is there a way that exists to paste back and use a command like that? Does a json representation bring value? (In QGIS you can copy and paste the command run to populate the box again, using a json format).

Not sure I understand, something like the --ui ?

For the history as json part, I assume it's available as this pending PR fixes when a comment is multiple lines #4216.
I'm wondering if copying here allows it to be used somewhere useful after that, or we would be better off to copy as json. If we copy as json, is there also a way to use that json somewhere?

[aniket2405]

@echoix, thanks for updating my pull request with the required change.
Are there any suggestions for this PR, or can we move forward and merge this?

[echoix]

I wanted to let the chance to get some responses, or for others to take a look at it if they objected. But I'm pretty sure it's cortect as is, and I spot checked a couple of wx calls to make sure their usage was fine, and "TheClipboard" seems fine and similar to how it's used elsewhere.

So, let's continue with it, it was well described in the PR description.

[aniket2405]

Thanks for the quick feedback and for merging this!