search

ggrossetie / asciidoctor-emoji

Asciidoctor.js extension to add emoji in your document!
MIT License
7 stars 5 forks source link

Possible format bug? #15

Closed mitm001 closed 4 years ago

mitm001 commented 4 years ago

Is this a bug with double colon?

https://github.com/Mogztter/asciidoctor-emoji/blob/59b603952c2f4fbf95b9dddac1416a17ed0e54a3/src/twemoji-map.js#L666

These symbols have broken image when used. I suspect it to be the extra spaces before and after minus signs. https://github.com/Mogztter/asciidoctor-emoji/blob/59b603952c2f4fbf95b9dddac1416a17ed0e54a3/src/twemoji-map.js#L1651-L1652

https://github.com/Mogztter/asciidoctor-emoji/blob/59b603952c2f4fbf95b9dddac1416a17ed0e54a3/src/twemoji-map.js#L1655-L1661

mitm001 commented 4 years ago

Broken due to spaces. https://github.com/Mogztter/asciidoctor-emoji/blob/59b603952c2f4fbf95b9dddac1416a17ed0e54a3/src/twemoji-map.js#L809-L812

mitm001 commented 4 years ago

Submitted p/r to fix issues. #16

ggrossetie commented 4 years ago

fixed in #16 thanks @mitm001 🎉