Created attachment 985
patch for Image.php for fixing size and aspect ratio of SVG images without width/height attributes.

attachment svgSize.diff ignored as obsolete

It looks like this would break if one but not both of $width and $height is
specified. These lines are doing arithmetic against the strings:

+ if ( !$width ) $width = $height ? ( $height * $f ) : 2048;
+ if ( !$height ) $height = $width / $f;

while the input values are only later normalized to pixel sizes.

Created attachment 1026
updated patch

I hope I fixed all issues now.

attachment svgSize.diff ignored as obsolete

Created attachment 1027
fixed float behavior

gave it another go...

Attached:

SVG 1.1 spec says viewBox values are "separated by whitespace and/or a comma", but
this seems to check whitespace only.

wfScaleSVGLength should probably just always return a float. If you want to check
for unset stuff, set the false ahead and check for the false to remain after *not*
setting the value different. There is for instance a running of wfScaleSVGLength()
on false values added in this patch, which should just leave the prior code
intact.

Better not to try looking for exact 0 values probably (especially if one's an int
and one's a float so they won't match!)

+ $f = $w / $h;
There's no guard against a float divide-by-zero; this may produce a runtime
warning. (The check for === 0 will fail if given a float 0.0.)

+ $width= round( $width );
bad spacing on these lines

• Bug 4609 has been marked as a duplicate of this bug. ***

Removing bogus dependency; this was for interpretation of the file contents by
MediaWiki.

• Bug 11163 has been marked as a duplicate of this bug. ***

Just noting that the example in bug 11163 has width="100%" height="100%" and requires the same sorts of viewBox hacks, presumably.

Added this w/ the new SVG size arch in r44651 / r44652.

Should also handle percentages on the width element reasonably, I hope. :)