On Thu, Feb 10, 2005 at 05:44:56PM -0600, Edward Flick wrote:
Hey guys, I'm altering SPImage to support the 'preserveAspectRatio' attribute as per the SVG spec for raster images (although when its set to 'slice' it will assume 'overflow' is set to hidden). I should finish with this tomorrow (hopefully). Does this sound like something that you guys would be interested in integrated into Inkscape CVS?
Hi Edward,
Post it to the patch tracker. It will definitely be applied unless there is something specifically wrong with it (in which case someone will give you some feedback).
Our policy has always been to keep a very open development process, so patches are essentially acceptable by default ('innocent until proven guilty'). Typical reasons that patches might not be accepted would be if the patch doesn't compile, invalidates a standard (such as SVG or HIG), or if there are firm plans to implement the feature in a different way. If the patch has a bug, sometimes we'll accept it anyway, if the bug is likely to get fixed, but more often we'll give the author a chance to repair it before it gets integrated.
It sounds like your change would _increase_ conformance to the SVG spec, and those kinds of patches are exceptionally valued. :-)
Bryce