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:
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?
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.inkscapefor um.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?