[BUG]: uppercase function names, angle units return `undefined`, throw error, respectively

[camdecoster]

Description

Passing in function names and angle units in uppercase causes culori to throw. Uppercase function names and angle units are allowed per the CSS spec.

Screenshots

Steps to reproduce

• Open the culori site
• Open the browser devtools console
• Enter culori.parse("RGB(128 0 0)")
• Note that undefined is returned
• Enter culori.parse("hsl(0DEG 100% 50%)")
• Note that an error is thrown

Notes

• The CSS values spec states that function names should be case-insensitive
• The CSS values spec states that unit identifiers should be case-insensitive

[camdecoster]

I'm happy to put together a PR to fix the issues if that would be helpful.

[danburzo]

Thanks @camdecoster, indeed we fail to account for character case in function and unit names. Unless I’m missing some other aspect where casing is relevant, I believe we could just add toLowerCase() after trim()?

culori/src/parse.js

Lines 145 to 146 in 435a1fc

	export function tokenize(str = '') {
	let chars = str.trim();

A PR for this, along with a couple of tests, would be appreciated, thanks!

[danburzo]

Oops, missed this ongoing PR:

• fix(parse): match color function names and keywords case-insensitively #275

[camdecoster]

Timely! I'm glad that it will be fixed soon.