Handle LWSP when parsing tts:fontFamily

mattjuggins · web-flow · commit 2c939c828aca · 2023-12-22T08:40:25.000-08:00
#250
diff --git a/src/main/js/styles.js b/src/main/js/styles.js
@@ -193,6 +193,7 @@
                 var rslt = [];
 
                 for (var i = 0; i < ffs.length; i++) {
+                    ffs[i] = ffs[i].trim();
 
                     if (ffs[i].charAt(0) !== "'" && ffs[i].charAt(0) !== '"') {
 
diff --git a/src/test/resources/unit-tests/fontFamily.ttml b/src/test/resources/unit-tests/fontFamily.ttml
@@ -0,0 +1,16 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<tt xml:lang="en" xmlns="http://www.w3.org/ns/ttml" xmlns:ttm="http://www.w3.org/ns/ttml#metadata" 
+xmlns:ttp="http://www.w3.org/ns/ttml#parameter" ttp:profile="http://www.w3.org/ns/ttml/profile/imsc1/text"
+xmlns:tts="http://www.w3.org/ns/ttml#styling" ttp:cellResolution="1 1" tts:extent="100px 100px">
+	<head>
+	</head>
+	<body>
+		<div>
+			<p tts:fontFamily="default">Maps 'default' generic family name to 'monospaceSerif' correctly.</p>
+			<p tts:fontFamily="Arial,default">Maps 'default' value used as a fallback to 'monospaceSerif' correctly.</p>
+			<p tts:fontFamily="Arial, default ">Maps 'default' value used as a fallback with spacing characters to 'monospaceSerif' correctly.</p>
+			<p tts:fontFamily="'My Test Font', default ">Fonts names wrapped in '' are preserved.</p>
+			<p tts:fontFamily='"My Test Font", default '>Fonts names wrapped in "" are preserved.</p>
+		</div>
+	</body>
+</tt>
diff --git a/src/test/webapp/js/unit-tests.js b/src/test/webapp/js/unit-tests.js
@@ -167,4 +167,46 @@ QUnit.test(
         return getIMSC1Document("unit-tests/metadataHandler.ttml", mh);
 
     }
-);
+);
+
+QUnit.test(
+    "Font Family Parsing",
+    function (assert) {
+
+        return getIMSC1Document("unit-tests/fontFamily.ttml").then(
+            function (doc) {
+
+                assert.deepEqual(
+                    doc.body.contents[0].contents[0].styleAttrs["http://www.w3.org/ns/ttml#styling fontFamily"],
+                    ["monospaceSerif"],
+                    "Maps 'default' generic family name to 'monospaceSerif' correctly."
+                );
+
+                assert.deepEqual(
+                    doc.body.contents[0].contents[1].styleAttrs["http://www.w3.org/ns/ttml#styling fontFamily"],
+                    ["Arial", "monospaceSerif"],
+                    "Maps 'default' value used as a fallback to 'monospaceSerif' correctly."
+                );
+
+                assert.deepEqual(
+                    doc.body.contents[0].contents[2].styleAttrs["http://www.w3.org/ns/ttml#styling fontFamily"],
+                    ["Arial", "monospaceSerif"],
+                    "Maps 'default' value used as a fallback with spacing characters to 'monospaceSerif' correctly."
+                );
+
+                assert.deepEqual(
+                    doc.body.contents[0].contents[3].styleAttrs["http://www.w3.org/ns/ttml#styling fontFamily"],
+                    ["'My Test Font'", "monospaceSerif"],
+                    "Fonts names wrapped in '' are preserved."
+                );
+
+                assert.deepEqual(
+                    doc.body.contents[0].contents[4].styleAttrs["http://www.w3.org/ns/ttml#styling fontFamily"],
+                    ['"My Test Font"', "monospaceSerif"],
+                    'Fonts names wrapped in "" are preserved.'
+                );
+            }
+        );
+
+    }
+);
