Last modified: 2011-07-15 19:12:13 UTC
Resizing an image with a width much smaller than height to a specified height is broken, because it may result in a zero pixel width. A minimum width of 1 pixel for a thumbnail should be enforced. See http://test.wikipedia.org/wiki/Bug_28076 to see the bug in action.
Did part of the patch you had on Bug #28076 fix this? If so, could you attach that patch here?
Created attachment 8475 [details] Broken patch It should have, but it broke some other cases which I haven't fully investigated. Parser tests indicate that in some cases the height attribute of <img>s is set to -1.
Copying comment from triage for posterity: patch needs to be checked to see if it changes any 'real' image sizes and to confirm that it correctly ensures minimum single pixels without becoming inconsistent (since we refer to thumbs by their width, we can't have a 1px-wide image that's 50px high and another that's 500px high.)
Yes, that would break. We'll have to output the 1px image then and defer to client side scaling for the further downscaling.
Related bug 24763
What is the status of this?
r92279
