feat: Allow to set width with autoresize

Fixes #91
StarCitizenWiki · May 4, 2024 · 21b6e96 · 21b6e96
1 parent 05575f8
commit 21b6e96
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/includes/EmbedService/EmbedHtmlFormatter.php b/includes/EmbedService/EmbedHtmlFormatter.php
@@ -29,7 +29,7 @@ final class EmbedHtmlFormatter {
 	 * description: String - Optional Description
 	 * @return string
 	 */
-	public static function toHtml( AbstractEmbedService $service, array $config = [] ): string {
+	public static function toHtml( AbstractEmbedService $service, array $config = [], array $args = [] ): string {
 		if ( $service instanceof OEmbedServiceInterface ) {
 			return self::makeIframe( $service );
 		}
@@ -60,6 +60,12 @@ public static function toHtml( AbstractEmbedService $service, array $config = []
 
 		if ( $config['autoresize'] === true ) {
 			$config['class'] .= ' embedvideo--autoresize';
+			if ( isset( $args['width'] ) ) {
+				$inlineStyles['container'] .= sprintf( 'max-width:%dpx', $width );
+			}
+			if ( isset( $args['height'] ) ) {
+				$inlineStyles['wrapper'] .= sprintf( 'max-height:%dpx', $height );
+			}
 		} else {
 			// Autoresize does not need inline width and height
 			$inlineStyles['container'] .= sprintf( 'width:%dpx', $width );

diff --git a/includes/EmbedVideo.php b/includes/EmbedVideo.php
@@ -314,7 +314,8 @@ public function output(): array {
 			// This does the whole HTML generation
 			EmbedHtmlFormatter::toHtml(
 				$this->service,
-				$this->makeHtmlFormatConfig( $this->service )
+				$this->makeHtmlFormatConfig( $this->service ),
+				$this->args
 			),
 			'noparse' => true,
 			'isHTML' => true