noviny / emoji-uid Goto Github PK

Oh no! There are no tests for this serious and important package? How could anyone forget that?

Initial release includes only one skin tone for all the people emojis. This is mostly due to this being kind of a silly side project that I wanted to not spend too much time on, but this irks the heck out of me.

Solution is to add all skin tone options for any skin-tone-modifiable emoji, but then also probably cut down the number of emojis that can have skin tones to make sure people-emojis are not too dominant in the spread of emojis.

This may be done tomorrow or never, depending on arbitrary inclination. If this irks you too, and want to take the time to fix it, that would be 🎁

Might want to add collision detection to make sure no two ids are same.

Eg:

const id = eid(1)

// If eid(1) is invoked a 100 times
// there's a good probability to receive the same id
// more than once.

Solution: Could store a local cache of all generated ids to retry when there's a collision.