User "Aaron Schulz" posted a comment on MediaWiki.r92279.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/92279#c22086
Commit summary:

(bug 28762) Resizing to specified height broken for very thin images

Restructure and comment of ImageHandler::normaliseParams(). 
ImageHandler::normaliseParams() will now output both width/height as 
physicalWidth/Height, where width/height are the dimensions used by the 
browser, and physicalWidth/Height are the dimensions of the thumbnail that will 
be created. This allows keeping the use of unique dimensions for a specified 
thumbnail width, by forcing client side resizing of 1px-wide images with non 
corresponding height.
Also fixed a bug where the check for mustRender() was in an if statement where 
it should not have been.

Comment:

<pre>
                                $swapDimensions = true;
+                               list( $params['width'], $params['height'] ) = 
+                                       array(  $params['width'], 
$params['height'] );
+                               list( $params['physicalWidth'], 
$params['physicalHeight'] ) = 
+                                       array( $params['physicalWidth'], 
$params['physicalHeight'] );
</pre>
What does this do?

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to