shortcode requires leading space?

[timmyg]

doesn't convert:

hey how are you:blush:

converts properly:

hey how are you :blush:

Any thoughts on how to handle this scenario? Didnt see anything in docs. Thanks!

[johanbrook]

Hi! It's currently not handled, but I can see the use case. PR is welcomed :)

[timmyg]

OK. What are your thoughts on best way to implement?

On Tue, Jun 14, 2016, 11:37 AM Johan Brook [email protected] wrote:

Hi! It's currently not handled, but I can see the use case. PR is welcomed
:)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#4 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABF1srjgXMiOnC9XK08eaijo0Jnd_gdiks5qLsq8gaJpZM4I1cxt
.

[johanbrook]

Probably tweaking the regex in here: https://github.com/lookback/meteor-emoji/blob/master/emojis.coffee#L3, but it'll be tricky, since we don't want it to parse things like http:// and make :/ into a smiley emoji. See my comment in the code.

Generally, I don't know if this is worth fixing.

[littke]

Slack doesn't support this either, just FYI. Maybe by design.

[johanbrook]

Probably.

(Test: foo:fire: )

Ok, GitHub handles it:

(Test: foo:fire: )