Live effects dont work in grouped shapes

[Jabiertxof]

Live effects dont work on grouped shapes, yes ungruped.

[Jabiertxof (jabiertxof)]

Shapes like star, box...

[Jabiertxof (jabiertxof)]

There are two? roads to make it.
One is make LPE objects paths, like in 0.48.
The other is not, this is better, but need a lot of changes, maybe not necesary...

[Jabiertxof (jabiertxof)]

For the second option, think i need a new lpe unction tath return the lpeitem whith haspatheffect() recursive.
And some changes in the LPE part on al shapes code

[Jabiertxof (jabiertxof)]

This fix contains:
1.- the patch https://bugs.launchpad.net/inkscape/+bug/1247324
2.- A fix to avoid strange results on apply on nested groups, now take the lowest effet in the tree.
3.- Now apply Live Effects to stars, ellipses an spirals

Thanks to Johan Engelen and su_v for his help.

[Jabiertxof (jabiertxof)]

And rects whith the help of Johan Engelen

[Jabiertxof (jabiertxof)]

Point 3 apply to shapes inside a group, the only requisite is you can only apply one LPE list effect to each element in the tree, taking the lowest effect, the nearst or just the end shape.

[Jabiertxof (jabiertxof)]

Fix bug in mixed group contents (path/shapes)

[Jabiertxof (jabiertxof)]

Patchs dont working well,

[Jabiertxof (jabiertxof)]

Only one stack of effects can be applied to a shape -paths,star,ellipse,spiral-. This remove the strange effects in stable and trunk whith nested groups and disctint LPE applied in the tree. For now the lowest effect are applied. For example the effect nearest or be a star inside nested tree of a group.

Included two patch: Convert to shapes || Apply as shapes

1. Convert to shapes (LPE_fixes_and_convert_V1.diff):
Convert star, ellipse, rect and spiral to a path shape it this are inside a group and dont have a previous LPE.
Now correct recalculation of bbox if nore than one effects are in the stack.

2. Apply as shapes (LPE_fixes_and_non_convert_V1.diff):
Some positioning bugs cause not perfect behaviour, disctint than shapes converted to paths.
Some effects dont work well, for example lattice2 render bad with this patch.

After put the two patch...

[Jabiertxof (jabiertxof)]

Apply as shapes (LPE_fixes_and_non_convert_V1.diff)

[Jabiertxof (jabiertxof)]

Convert to paths (LPE_fixes_and_convert_V1.diff).

[Jabiertxof (jabiertxof)]

Sorry for the updates of pach. Now start versioning but i want a solid one base.

[Jabiertxof (jabiertxof)]

New version 2 of the patch.
Now has the ability to apply effects in diferent levels of the tree (group tree)
Again 2 roads:

1.- Apply as shapes (LPE_fixes_and_non_convert_V2.diff):
Some BBox problems.
Star, Ellipse and Spiral whith a LPE dont be affected by group LPE.

2.- Convert to paths (LPE_fixes_and_convert_V2.diff)
No bugs detected at now.

Files soon...

[Jabiertxof (jabiertxof)]

Apply as shapes (LPE_fixes_and_non_convert_V2.diff)

[Jabiertxof (jabiertxof)]

Convert to paths (LPE_fixes_and_convert_V2.diff)

[Jabiertxof (jabiertxof)]

New patch. Works the same but remove extra staff not necesary from non convert patch
Convert to paths (LPE_fixes_and_convert_V3.diff)

[Jabiertxof (jabiertxof)]

More cleanup

[Jabiertxof (jabiertxof)]

Fixed in r.a113f466 of Master