search

tinted-theming / home

Style systems and smart build tooling for crafting high fidelity color schemes and easily using them in all your favorite apps.
MIT License
252 stars 12 forks source link

Fix missing Lowercase instruction in Slugify #87

Closed titaniumtraveler closed 8 months ago

titaniumtraveler commented 8 months ago

I'm currently working on updating the rust crate base16_color_scheme from the old Base 16 spec to this one. While implementing the slugify functionality I found that the instruction to lowercase the characters was missing. This pr fixes it.

Also the line Drop all non-alphanumeric and non-dash characters is a bit ambiguously worded and something like Drop all characters that are neither alphanumeric nor dashes might be better. \ Though that is only a minor nitpick. (I added it as separate commit for convenience.)