-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Re-add emoji detection script #7440
Comments
So did some testing to determine the current status of this. With Gutenberg as is now, the emojis will paste but not appear. They are saved as emojis in the database, though. When I revert #2799, the following issues occur:
Screencast of my testing: https://cloudup.com/cd4i2kGPMLu |
Twemoji could be updated in Core. Core is on 11.0, the latest version is 11. with two small updates in between. I tested this locally, but it does not solve any of the issues described in my last comment. |
We cannot simply re-add the emoji script while also using React and the RichText component.
|
I would need this for replacing an emoji character (shape provided by font) by a SVG image for precisely styling some emojis (by theme). |
Hey @kraftbj Brandon. Could you please retest using WP 5.7 beta and Gutenberg plugin 10. |
I tested this issue with the latest Gutenberg trunk.
As far as I've tested, none of these issues occur and everything appears to be working as expected. Therefore, I would like to close this issue, but please feel free to reopen it at any time if I have missed anything. f65c9184b4d4179b65575e522a35f44d.mp4 |
Describe the bug
In #6151, the emoji detection script was removed to avoid a conflict with Gutenberg saving emoji-as-images. I didn't see a corresponding issue to track fixing the underlying issue and re-enabling emoji detection support.
This will generally be needed for 5.0 and Trunk now included Twemoji 11—which adds the new emoji from Unicode 11 (cupcakes! superheroes!).
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The cupcake emoji (on an OS that does not yet support a cupcake, macOS 10.13.5 for example) should render.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Smartphone (please complete the following information):
The text was updated successfully, but these errors were encountered: