add curly braces when attribute shorthand is used but not allowed, fixes

sveltejs#110
jesseskinner · Aug 18, 2020 · adb6c93 · adb6c93
1 parent c7446d3
commit adb6c93
Showing 1 changed file with 15 additions and 12 deletions.
diff --git a/src/print/index.ts b/src/print/index.ts
@@ -229,18 +229,21 @@ export function print(path: FastPath, options: ParserOptions, print: PrintFn): D
 
             if (isAttributeShorthand && options.svelteAllowShorthand) {
                 return concat([line, '{', node.name, '}']);
-						} else {
-							const def: Doc[] = [line, node.name];
-							if (node.value !== true) {
-									def.push('=');
-									const quotes = !hasLoneMustacheTag || options.svelteStrictMode;
-
-									quotes && def.push('"');
-									def.push(...path.map(childPath => childPath.call(print), 'value'));
-									quotes && def.push('"');
-							}
-							return concat(def);
-						}
+			} else {
+                const def: Doc[] = [line, node.name];
+                if (node.value !== true) {
+                    def.push('=');
+                    const quotes = !hasLoneMustacheTag || options.svelteStrictMode;
+                    const curlies = quotes && isAttributeShorthand && !options.svelteAllowShorthand;
+
+                    quotes && def.push('"');
+                    curlies && def.push('{');
+                    def.push(...path.map(childPath => childPath.call(print), 'value'));
+                    curlies && def.push('}');
+                    quotes && def.push('"');
+                }
+                return concat(def);
+            }
         }
         case 'MustacheTag':
             return concat(['{', printJS(path, print, 'expression'), '}']);