On Sun, Jul 23, 2006 at 08:43:24AM -0500, Aaron Spike wrote:
Aaron Spike wrote:
Bryce Harrington wrote:
I've gone ahead and committed this. If the virtual thing is the only issue, and it otherwise looks okay, it's probably better to have it in the codebase where people can start testing it. Looks like Aaron is creating a script to take advantage of this fix to address the marker coloring bug, so this'll get him able to test that.
Well, my script runs well enough from the command line, but fails miserably when called from the Effects menu. I actually looks like there is code written specificly to prevent editing the defs via effects. See http://inkscape.modevia.com/doxygen/html/classInkscape_1_1Extension_1_1Imple...
BTW, when I comment out those 4 lines the script works to change a markers to match a stroke with a solid color. I'm not sure what happens with gradients.
I'm sure ted had a rationale for why to hide defs, but if your tests with this patch don't display any untoward behavior among the various scripts, then maybe we should commit it, and see what misbehaviors actually turn up in practice. Perhaps there is a less brute force way to handle those issues. I think being able to use scripts to vary defs would enable a lot of handy new extensions.
Bryce