Skip to content
This repository has been archived by the owner on Oct 3, 2018. It is now read-only.

Emoji not found in <p>:)</p> #57

Closed
wants to merge 3 commits into from
Closed

Emoji not found in <p>:)</p> #57

wants to merge 3 commits into from

Conversation

adam-lynch
Copy link
Contributor

Fails because of #51

@hassankhan hassankhan changed the title Added test for <p>:)</p> Emoji not found in <p>:)</p> Sep 21, 2014
@hassankhan hassankhan added the bug label Sep 22, 2014
@adam-lynch
Copy link
Contributor Author

This not just short emoji, it is all emoji. Just discovered that.

@adam-lynch
Copy link
Contributor Author

Also, in case it's not obvious, this is a massive problem for markdown users; hello is converted to <p>hello</p> first.

@hassankhan
Copy link
Contributor

Could you possibly add another test to the same branch highlighting that too?

@adam-lynch
Copy link
Contributor Author

Sure, I'll do that sometime today.

@hassankhan
Copy link
Contributor

Cheers mate 😄

@adam-lynch
Copy link
Contributor Author

@hassankhan done

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants