[Patch] Size of tagged nodes no longer configureable

[petschge]

Since changeset 3164 the size of tagged nodes can no longer be set via mappaint.node.tagged-size in the configuration. Please reintroduce this.

A way to set color and/or filling for junction nodes would be nice too.

If you want to be really thorough verify that size, color and filling for selected, unselected, tagged and junction nodes can be configured individually.

[petschge]

The following patch should fix the issue but somehow doesn't

[bastiK]

The properties (a) size, (b) color and (c) filled or not should be orthogonal, somehow: If a tagged node is filled and a connection node is 5 px, then a tagged connection node should be both filled and larger than a default node.

[petschge]

Fix a bug from the first version of the patch and add support for the MapPaint Style.

[petschge]

Node size should probably be the maximum of all the applicable node sizes. If any of the applicable fill properties is the the node should probably be filled. Color is quite difficult though.

[bastiK]

Would be best if a color could be set as default, i.e. then it equals the normal node color. If both, the junction color and the tagged color are different from default, there must be a precedence.

[petschge]

Yeah, precedence of junction color vs tagged color is really the only case we have to solve as selection color should always override the other colors. Personally I don't really care which one takes precedence but would suggest that tagged color overrides junction color (if it differs from the normal node color) as I feel that "junction" is less important then "tagged".

[petschge]

Add color, filling and size logic as per the discussion in this tiket. Tested and works well for me. Please commit upstream.

[bastiK]

(In [3180]) applied #4876 - make size of tagged nodes configurable (patch by petschge)

[malenki]

for the record:
I guess it was this patch that re-set the nodes to the standard color (yellow) in latest, that is, 3184.

[anonymous]

The commiter changed "mappaint.node.fill-junction" to "mappaint.node.fill-onnection" which (in principle) is fine, but is lacking a "c" in the name. Guess this should be fixed ASAP.

Same thing happend to "mappaint.node.onnection-size".

[bastiK]

(In [3185]) fixed typo (see #4876 - Size of tagged nodes no longer configureable)

[bastiK]

Thanks for pointing that out.