TZP: "script defaults" metric catches a "fontList.match(...) is null" error

[Thorin-Oakenpants]

Prerequisites

Please use issues for bugs only! Answer the following questions for yourself before submitting an issue: YOU MAY DELETE THE PREREQUISITES SECTION.

• I am running the latest version
• I checked the documentation and found no answer
• I checked to make sure that this issue has not already been filed

Expected Behavior

no errors: values are defaults for: proportional-font + monospace size + sans-serif size + serif size

as seen in Firefox> Settings > Language and Settings > Fonts > Advanced > per script in the dropdown

• for example here is devanagari which is sans-serif-13-16-16 as per the expected example above

Current Behavior

https://arkenfox.github.io/TZP/tzp.html#fonts > script defaults metric

• "TypeError: can't access property "map", fontList.match(...) is null"

• chameleon/src/lib/spoof/font.ts

  Line 73 in 03e3a91

  let tmpFonts = fontList.match(REGEX_FONTS).map(f => f.replace(/,+$/, ""));

• code used
  • basically it loops a bunch of writing scripts by setting a language tag on an element, and for each of those loops three generic font-families (monospace, serif, sans-serif). I actually use two elements for technical reasons, but that's not important

Relevant settings

enable Spoof font fingerprint

Context (Environment)

all of them

[sereneblue]

Thanks, I'll review.

[Thorin-Oakenpants]

Thanks, I'll review.

🍕