Comment 29 for bug 171117

Revision history for this message
Robin Battey (zanfur) wrote :

It performs exactly as your original one does, except faster (just tested). It also performs exactly how simply ungrouping manually would work -- I get the same results that I get when I select everything and simply ungroup manually (it takes 7 ungroups). Do you intend for your plugin to have different behavior than simply running the ungroup command a bunch of times?

The problem is that the clip path is not preserved when ungrouping, see here:

  http://www.inkscapeforum.com/viewtopic.php?f=5&t=5682

It could be changed recognize a clip and not ungroup that, or to transform the clipped object so that the clip is no longer necessary. I think the former is better. But, that's a functional change -- the cleanup didn't alter the functionality in any way that I can see.

I also see that deep ungroup doesn't play well with inkscape undo. Is that something easily fixable?