TheDJ has uploaded a new change for review.
https://gerrit.wikimedia.org/r/278574
Change subject: Remove double normalization of audio player width
......................................................................
Remove double normalization of audio player width
This is duplicat with the check done by normaliseParams.
Change-Id: I324c0892a03e2e3c23209f2a5de2f79f00274d94
---
M TimedMediaHandler_body.php
1 file changed, 0 insertions(+), 6 deletions(-)
git pull
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TimedMediaHandler
refs/changes/74/278574/1
diff --git a/TimedMediaHandler_body.php b/TimedMediaHandler_body.php
index e5d0c70..93fe385 100644
--- a/TimedMediaHandler_body.php
+++ b/TimedMediaHandler_body.php
@@ -364,12 +364,6 @@
$srcWidth = $file->getWidth();
$srcHeight = $file->getHeight();
- // Audio should not be transformed by size, give it a default
width and height
- if ( $this->isAudio( $file ) ) {
- $srcWidth = 220;
- $srcHeight = 23;
- }
-
$params['width'] = isset( $params['width'] ) ? $params['width']
: $srcWidth;
// if height overtakes width use height as max:
--
To view, visit https://gerrit.wikimedia.org/r/278574
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I324c0892a03e2e3c23209f2a5de2f79f00274d94
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TimedMediaHandler
Gerrit-Branch: master
Gerrit-Owner: TheDJ <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
