fix: handle double quoted fontNames for fontFamily/css (#1575)

Pythonen · web-flow · commit dbcdae35739f · 2025-10-01T12:14:41.000+10:00
* fix: handle double quoted fontNames

* chore: add changeset
diff --git a/.changeset/tasty-ligers-try.md b/.changeset/tasty-ligers-try.md
@@ -0,0 +1,5 @@
+---
+'style-dictionary': patch
+---
+
+Fix fontName parsing to handle double quotes
diff --git a/__tests__/common/transforms.test.js b/__tests__/common/transforms.test.js
@@ -1305,6 +1305,15 @@ describe('common', () => {
         ).to.equal('Arial, sans-serif');
       });
 
+      it('should handle double quoted fontFamily values', () => {
+        expect(
+          fontFamilyTransform({
+            value: 'Some Font, "With Double Quotes", \'And Single Quotes\'',
+            type: 'fontFamily',
+          }),
+        ).to.equal("'Some Font', \"With Double Quotes\", 'And Single Quotes'");
+      });
+
       it('should wrap fontFamily values with spaces in quotes', () => {
         expect(fontFamilyTransform({ value: 'Gill Sans', type: 'fontFamily' })).to.equal(
           "'Gill Sans'",
diff --git a/lib/common/transforms.js b/lib/common/transforms.js
@@ -121,7 +121,9 @@ function getBasePxFontSize(config) {
  */
 function quoteWrapWhitespacedFont(fontString) {
   let fontName = fontString.trim();
-  const isQuoted = fontName.startsWith("'") && fontName.endsWith("'");
+  const isQuoted =
+    (fontName.startsWith("'") && fontName.endsWith("'")) ||
+    (fontName.startsWith('"') && fontName.endsWith('"'));
   if (!isQuoted) {
     fontName = fontName.replace(/'/g, "\\'");
   }

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'style-dictionary': patch
 +---
++
 +Fix fontName parsing to handle double quotes