Reuse the loc of the macroMaybeAttrs hashes for the created attributes

The newer template compiler sorts attributes based on their `loc` location when printing. Since previously no `loc` info was added to the newly created attributes, they were being moved to the front of the element. By reusing their original `loc` data we can ensure that the order from the source file is maintained.
embroider-build · Feb 4, 2022 · 4951e11 · 4951e11
1 parent a3fef30
commit 4951e11
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/macros/src/glimmer/macro-maybe-attrs.ts b/packages/macros/src/glimmer/macro-maybe-attrs.ts
@@ -20,11 +20,11 @@ export function maybeAttrs(elementNode: any, node: any, builders: any) {
 
   if (result.value) {
     for (let bareAttr of bareAttrs) {
-      elementNode.attributes.push(builders.attr(bareAttr.original, builders.text('')));
+      elementNode.attributes.push(builders.attr(bareAttr.original, builders.text(''), bareAttr.loc));
     }
 
     for (let attr of node.hash.pairs) {
-      elementNode.attributes.push(builders.attr(attr.key, builders.mustache(attr.value)));
+      elementNode.attributes.push(builders.attr(attr.key, builders.mustache(attr.value), attr.loc));
     }
   }
 }