fix: preserve whitespace before a * namespace in attribute selectors (#325)

The `*` branch of the attribute parser stored the raw leading whitespace
from `spaceBefore`, but that variable had just been cleared a few lines
above, so `raws.spaces.attribute.before` became empty and `[ *|a ]`
round-tripped to `[*|a ]`. The word-namespace branch already uses
`commentBefore` here; do the same for `*`.

Author: spokodev

src/__tests__/attributes.mjs

@@ -28,6 +28,13 @@ test("attribute selector spaces with namespace (both)", "[  foo|bar   ]", (t, tr
   t.deepEqual(tree.nodes[0].nodes[0].type, "attribute");
   t.falsy(tree.nodes[0].nodes[0].quoted);
 });
+test("attribute selector spaces with universal namespace (both)", "[  *|bar   ]", (t, tree) => {
+  t.deepEqual(tree.nodes[0].nodes[0].namespace, "*");
+  t.deepEqual(tree.nodes[0].nodes[0].attribute, "bar");
+  t.deepEqual(tree.nodes[0].nodes[0].spaces.attribute.before, "  ");
+  t.deepEqual(tree.nodes[0].nodes[0].spaces.attribute.after, "   ");
+  t.deepEqual(tree.nodes[0].nodes[0].type, "attribute");
+});
 test("attribute selector spaces (both)", "[  href   ]", (t, tree) => {
   t.deepEqual(tree.nodes[0].nodes[0].attribute, "href");
   t.deepEqual(tree.nodes[0].nodes[0].spaces.attribute.before, "  ");

src/parser.js

@@ -228,7 +228,7 @@ export default class Parser {
             }
             if (commentBefore) {
               ensureObject(node, "raws", "spaces", "attribute");
-              node.raws.spaces.attribute.before = spaceBefore;
+              node.raws.spaces.attribute.before = commentBefore;
               commentBefore = "";
             }
             node.namespace = (node.namespace || "") + content;