Add option to change tools on the command line

[pascal-niklaus]

This refers to #187

Changes:

(1) Command-line option added that allows changes of the tools while gromit is running, following the syntax of the config file.
Example:

  gromit-mpx --change-tool '"default"=PEN(color="green" size=4 arrowsize=2)'

(2) Updated man page to include this option, and also added documentation for --line that was missing

(3) Implementation details: a new file parser.c/.h was created that contains functions to scan the pen definitions. These functions are now called from config.c and from the callback invoked when changing these setting on-the-fly when gromit is running.

[bk138]

I have some comments at the respective code lines and some general thoughts.

The general ones:

• So this is essentially the meta-tool called toolswitcher devised in Make the various tools selectable by the system tray icon and from control #91 plus that it can redefine tools and minus that it's CLI only?
• My general goal is to make the app easy for non-pro users to use, so Add UI for creating/editing tools and mapping to buttons and/or maybe keys #110 is the go-to solution for a user interface; also the app should be usable w/o any third-party tools like key-mappers and the like. So, in general, I would not add too much special CLI features unless they're small and easy to maintain. So I guess I would merge this if you can slim it down (removing the refacto into parse.c e.g.), otherwise rather not. In any case, please don't be put off, I very much appreciate any contribution!

[pascal-niklaus]

I have some comments at the respective code lines and some general thoughts.

The general ones:

* So this is essentially the meta-tool called toolswitcher devised in [Make the various tools selectable by the system tray icon and from control #91](https://github.com/bk138/gromit-mpx/issues/91) plus that it can redefine tools and minus that it's CLI only?

Yes, this sounds a bit like it, except for the absence of a tray icon part.

* My general goal is to make the app easy for non-pro users to use, so [Add UI for creating/editing tools and mapping to buttons and/or maybe keys #110](https://github.com/bk138/gromit-mpx/issues/110) is the go-to solution for a user interface; also the app should be usable w/o any third-party tools like key-mappers and the like. So, _in general_, I would not add too much special CLI features _unless_ they're small and easy to maintain. So I guess I would merge this if you can slim it down (removing the refacto into parse.c e.g.), otherwise rather not. In any case, please don't be put off, I very much appreciate any contribution!

I can easily move the content of parser.c into config.c, and call these functions from main.c for the command-line option. That part is straightforward.

Additional thoughts:

One of my use cases is with a graphics tablet and presentations in full-screen mode, with no tray icon visible. With extra tools (colors, rect, line, single and double arrows, etc) at some stage the number of modifier keys is quickly exhausted.

Key mappings would work, but then it would be great to be able to distinguish keystrokes from an external keyboard (like a numeric keypad for tool changes) and the "normal" keyboard. That would be along the line with the option to have two mouse cursors, one for gromit (driven by e.g. a tablet pen) and another one for regular work.

[bk138]

Regarding this one, I think a CLI toolswitcher is something we would like to have.

Not sure if a tool-redefiner would add too much extra complexity. I'd rather have one single place (the .cfg or later on, the .ini file after #110) where definitions are stored and read from for switching/drawing purposes.

[pascal-niklaus]

Regarding this one, I think a CLI toolswitcher is something we would like to have.

Not sure if a tool-redefiner would add too much extra complexity. I'd rather have one single place (the .cfg or later on, the .ini file after #110) where definitions are stored and read from for switching/drawing purposes.

Another way to think about this is that the diversity of tools that gromit offers will keep growing (at least I have some more ideas in that direction ;-) ). Given this, I think it would be good to allow changing the individual attributes of the tools separately (e.g. tool type, color, line width etc).

For example, I may wish to draw in yellow, and then change from a PEN to a RECT or LINE (all still yellow). Later, maybe I want to change the color to green, or just modify the line width (again, preserving these choices for the future tools).

With individual tool definitions, an enormous variety of tools would need to be defined to accommodate all combinations (and one would have to remember which is which).

I understand that this is a different approach than providing a set of a few carefully chosen predefined tools, like this is now implemented with the modifier keys.

For the CLI, my suggestion is therefore to provide an option to change the individual attributes of the default tool.

This is based on my experience with a screenshot annotation tool I frequently use, ksnip. ksnip allows changing tools with a single key press (e.g. L = line, A= arrow, D = double arrow, R = rect etc), which in principle allows for a very smooth workflow. However, in each and every tool the color and line width has to be set separately, which IMO spoils the user experience.

But maybe the two approaches could be combined? For example, some tools could be defined in the .ini file and selected by their number, and the default tool attributes changed as outlined above?

[pascal-niklaus]

I have now integrated the content of "parser.c/parser.h" into config.c/config.h, as suggested.

Now, two command-line options are available for tool changes:

1. Using --change-tool or -T, a complete tool definition can be changed, for example like so:

gromit-mpx --change-tool '"default"=PEN(color="Limegreen" size=4 arrowsize=2 arrowtype="end")'

2. Using --change-attribute or -A an individual attribute of a tool can be changed, keeping all others. This way, it for example is possible to change from PEN to LINE and keep color, width, arrows etc, or to just change the color and line width, regardless of the tool. Examples are

gromit-mpx --change-attribute '"default"=(color="Limegreen" size=4)'   # change just attributes keeping tool
gromit-mpx --change-attribute '"default"=PEN(color="cyan")'            # change tool and some attributes
gromit-mpx --change-attribute '"default"=RECT'                         # change just the tool

[bk138]

After some thinking, I'm OK with the tool definition on the CLI.

Not sure about the attribute changer, is it really that useful that it justifies extra code, i.e. isnt't that already covered by the tool redefinition functionality?

Also, am a bit afraid of the vast refactor of config.[ch], but I guess this is needed to be able to parse config from other places in the program as well... :-)

[bk138]

manpage says "The result must be freed through gdk_rgba_free()." Should be added to paint-context_free() then.

[pascal-niklaus]

Now all changes requested should be addressed, and the PR be compatible with the current master.

Re your question:

• code refactoring in config: I agree that moving the config code to these extra functions looks like a relatively big change. OTOH, the code largely is the same, and IMO it makes the main config function much more readable (the loops in there were very long before).

• about changing the attributes of tools: this is very little extra code since it builds on the same functions. And I think it allows to mirror the "non-digital" drawing experience better. For example, one changes to a pen with a different color or width, and then draws a line, or a rectangle. With just complete tool changes, the rectangle would for example always be yellow and have thick lines, and the line be red. Binding color changes and tool changes to different keystrokes (in the desktop environment or in some other ways) seems to allow for a more natural drawing experience, IMO. For example, one could switch to yellow, and then draw lines, rectangles etc, and then change to green and do the same. Or use rectangles and then just change the colors.

[bk138]

Thank you very much - almost there!

[bk138]

That comment should be moved down to parse_config again.

[bk138]

We still have the allocated GdkRGBA here which should be freed.

[bk138]

@pascal-niklaus what do you think about those?