New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Emoji shortcode #2748
Emoji shortcode #2748
Conversation
@@ -0,0 +1,46 @@ | |||
# -*- coding: utf-8 -*- | |||
# This file is public domain according to its author, Brian Hsu |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leftover 1
|
||
|
||
class Plugin(ShortcodePlugin): | ||
"""Plugin for gist directive.""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leftover 2
Looks good, but perhaps we could merge all those JSON files into a more compact, manageable format of |
@Kwpolska yeah, I could merge them but then I would need to do it again if I ever refresh that data. |
@ralsina I didn’t expect you to do it by hand. |
Also, there are two turkey emojis! (bird and country, I guess) |
@@ -0,0 +1,21 @@ | |||
The MIT License (MIT) | |||
|
|||
Copyright (c) 2016 -2017 Shayan Rais |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a link to the repo somewhere.
Done. Also added docs.
…On Tue, May 9, 2017 at 10:33 AM Chris Warrick ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In nikola/plugins/shortcode/emoji/data/LICENSE
<#2748 (comment)>:
> @@ -0,0 +1,21 @@
+The MIT License (MIT)
+
+Copyright (c) 2016 -2017 Shayan Rais
Add a link to the repo <https://github.com/shanraisshan/EmojiCodeSheet>
somewhere.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2748 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAAGK71ChbHD3kxXKCBqfggeSpW0rSKgks5r4GsjgaJpZM4NUGe4>
.
|
output = '''<span class="emoji">{}</span>'''.format(TABLE[name]) | ||
except KeyError: | ||
LOGGER.warning('Unknown emoji {}'.format(name)) | ||
output = '''<span class="emoji, error">{}</span>'''.format(name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No comma here.
Pull Request Checklist
Description
A simple shortcode for emoji support: {{% emoji cry_face %}}