Merge pull request sveltejs#71 from sveltejs/sveltejsgh-63

handle empty attributes in elements and components

compiler/generate/visitors/attributes/addComponentAttributes.js

@@ -16,6 +16,13 @@ export default function addComponentAttributes ( generator, node, local ) {
 				});
 			}
 
+			else if ( attribute.value.length === 0 ) {
+				local.staticAttributes.push({
+					name: attribute.name,
+					value: `''`
+				});
+			}
+
 			else if ( attribute.value.length === 1 ) {
 				const value = attribute.value[0];
 

compiler/generate/visitors/attributes/addElementAttributes.js

@@ -31,6 +31,18 @@ export default function addElementAttributes ( generator, node, local ) {
 				}
 			}
 
+			else if ( attribute.value.length === 0 ) {
+				if ( propertyName ) {
+					local.init.push( deindent`
+						${local.name}.${propertyName} = '';
+					` );
+				} else {
+					local.init.push( deindent`
+						${local.name}.setAttribute( '${attribute.name}', '' );
+					` );
+				}
+			}
+
 			else if ( attribute.value.length === 1 ) {
 				const value = attribute.value[0];
 

test/compiler/attribute-empty-svg/_config.js

@@ -0,0 +1,7 @@
+export default {
+	html: `
+		<svg>
+			<g class=''></g>
+		</svg>
+	`
+};

test/compiler/attribute-empty-svg/main.html

@@ -0,0 +1,3 @@
+<svg>
+	<g class=''></g>
+</svg>

test/compiler/attribute-empty/_config.js

@@ -0,0 +1,3 @@
+export default {
+	html: `<div class=""></div>`
+};

test/compiler/attribute-empty/main.html

@@ -0,0 +1 @@
+<div class=''></div>

test/compiler/component-data-empty/Widget.html

@@ -0,0 +1 @@
+<p>foo: '{{foo}}'</p>

test/compiler/component-data-empty/_config.js

@@ -0,0 +1,3 @@
+export default {
+	html: `<div><p>foo: ''</p></div>`
+};

test/compiler/component-data-empty/main.html

@@ -0,0 +1,11 @@
+<div>
+	<Widget foo=''/>
+</div>
+
+<script>
+	import Widget from './Widget.html';
+
+	export default {
+		components: { Widget }
+	};
+</script>