Coverage decreased (-0.1%) to 99.485% when pulling 7f0b4b4 on evilebottnawi:issue-430 into cd92638 on ben-eb:master.

Maybe add monospace to keywords? I don't think the comment applies anymore. Also, I think we can also address #434 easily by making any checks against keywords case insensitive.

@andyjansson let's do it 👍

FYI: for test https://mothereff.in/font-family

Maybe add monospace to keywords

In code we have comment:

// Note that monospace is missing intentionally from this list; we should not
// remove instances of duplicated monospace keywords, it causes the font to be
// rendered smaller in Chrome.

But i will fix it anyway 👍

@andyjansson can you give me invite i close some invalid issues (with tests if needed)

In code we have comment:

// Note that monospace is missing intentionally from this list; we should not
// remove instances of duplicated monospace keywords, it causes the font to be
// rendered smaller in Chrome.

Yeah, I know. I don't think the comment applies anymore. I modified your code to include monospace in the keywords collection and the should not dedupe monospace and should not dedupe monospace (2) tests pass just the same.

@andyjansson hm, let's do it

/cc @andyjansson done, also fix some cases with lowercase/uppercase with tests