Merge pull request #958 from EricRihlmann/video-embed

Minor Video Embed Optimization

library/foundation.php

@@ -93,7 +93,7 @@ function foundationpress_active_list_pages_class( $input ) {
 endif;
 
 /**
- * Enable Foundation responsive videos for oEmbed
+ * Enable Foundation responsive embeds for WP video embeds
  */
 
 if ( ! function_exists( 'foundationpress_responsive_video_oembed_html' ) ) :
@@ -115,12 +115,13 @@ function foundationpress_responsive_video_oembed_html( $html, $url, $attr, $post
 
 	$is_video = false;
 
-	// Determine if oEmbed is a video
+	// Determine if embed is a video
 	foreach ( $video_sites as $site ) {
 		// Match on `$html` instead of `$url` because of
 		// shortened URLs like `youtu.be` will be missed
 		if ( strpos( $html, $site ) ) {
 			$is_video = true;
+			break;
 		}
 	}
 
@@ -141,14 +142,15 @@ function foundationpress_responsive_video_oembed_html( $html, $url, $attr, $post
 
 		$class = 'responsive-embed'; // Foundation class
 
-		// Determine if `widescreen`
+		// Determine if aspect ratio is 16:9 or wider
 		if ( is_numeric( $width ) && is_numeric( $height ) && ( $width / $height >= 1.7 ) ) {
 			$class .= ' widescreen'; // space needed
 		}
 
+		// Wrap oEmbed markup in Foundation responsive embed
 		return '<div class="' . $class . '">' . $html . '</div>';
 
-	} else { // not a supported video oEmbed
+	} else { // not a supported embed
 		return $html;
 	}