Closes #206 - Omitted sizes should be evaluated to 100vw per the …

…spec; using `width: 100%` of viewport for the sake of older browsers.
scottjehl · Apr 22, 2014 · 437097b · 437097b
1 parent 9f9555a
commit 437097b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/dist/picturefill.js b/dist/picturefill.js
@@ -224,11 +224,9 @@ window.matchMedia || (window.matchMedia = function() {
 	 */
 	pf.getCandidatesFromSourceSet = function( srcset, sizes ) {
 		var candidates = pf.trim( srcset ).split( /,\s+/ );
+		var widthInCssPixels = sizes ? pf.findWidthFromSourceSize( sizes ) : "100%";
 		var formattedCandidates = [];
-		var widthInCssPixels;
-		if ( sizes ) {
-			widthInCssPixels = pf.findWidthFromSourceSize( sizes );
-		}
+
 		for ( var i = 0, len = candidates.length; i < len; i++ ) {
 			var candidate = candidates[ i ];
 			var candidateArr = candidate.split( /\s+/ );

diff --git a/dist/picturefill.min.js b/dist/picturefill.min.js
diff --git a/src/picturefill.js b/src/picturefill.js
@@ -175,11 +175,9 @@
 	 */
 	pf.getCandidatesFromSourceSet = function( srcset, sizes ) {
 		var candidates = pf.trim( srcset ).split( /,\s+/ );
+		var widthInCssPixels = sizes ? pf.findWidthFromSourceSize( sizes ) : "100%";
 		var formattedCandidates = [];
-		var widthInCssPixels;
-		if ( sizes ) {
-			widthInCssPixels = pf.findWidthFromSourceSize( sizes );
-		}
+
 		for ( var i = 0, len = candidates.length; i < len; i++ ) {
 			var candidate = candidates[ i ];
 			var candidateArr = candidate.split( /\s+/ );