Hi Tonny,

   Two things.  First off I can see where the change goes int
SVGOMViewElement, but I don't see that SVGOMUseElement has a
getPreserveAspectRatio method (although I suppose this could
be a bug in the spec).

   Also can you provide a testcase for this?  The simplest thing
to do would be to write an SVG file with some embedded javascript
that uses this interface to populate some text elements in the
document.  This can then be tied into our regression testing
facilities to ensure it continues to do the right thing down
the line.  If you wanted to test update handling as well
(if you set the property through setAttribute and then query
via the PAR interface do you get the correct updated values) that
would be great.

Tonny Kohar wrote:

Hi,


   I will try and look at this shortly.  Updating to the new
License would be good.  Thanks!

I just upload the new file to bugzila id: 30580. It is the same file just change the header to Apache License 2.

When the patch is accepted I also would like to update
- SVGOMUseElement
- SVGOMViewElement
for preserveAspectRatio support. It is only a simple 1 line change.
Do I need to upload these files as well?

Best Regards
Tonny Kohar


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to