-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
replace emojione #20
Comments
Feature/amex credit card
how do we still feel about this? i think it adds to odd appeal of the site that we have some outdated emoji set that no one uses anymore. If anything, I'd like to have an emoji selection menu that displays all of these (like in discord or other chat dealies) |
I'm not sure how I feel, it does feel kind of part of the site now that its been around so long. If there's any bugs in it I'm not sure how we will upgrade, I forked it a long time ago to add custom emoji and haven't incorporated any upstream changes since. I guess the project isn't even called emojione anymore. |
If there were any serious bugs in it, I'd think they would have surfaced by now. Maybe something will break in the future, but it seems to me that it has been working just fine. I'd be inclined to close this and revisit the issue if emojione causes any issues. |
Sounds good to me |
If there was something that just used the normal system emoji images, with fallback to images, I think it would be better.
The text was updated successfully, but these errors were encountered: