Skip to content
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

Many Emoji Don't Render in SVG #18

Closed
BooDoo opened this issue Dec 20, 2022 · 4 comments
Closed

Many Emoji Don't Render in SVG #18

BooDoo opened this issue Dec 20, 2022 · 4 comments
Assignees
Labels
BACKEND BRIDGE for the SEND_STATUS submodule help wanted Extra attention is needed

Comments

@BooDoo
Copy link
Owner

BooDoo commented Dec 20, 2022

Reported on behalf of @[email protected]

Example Post

@BooDoo BooDoo added BACKEND svg2png unexpected behavior from svg2png labels Dec 20, 2022
@BooDoo BooDoo self-assigned this Dec 20, 2022
@BooDoo
Copy link
Owner Author

BooDoo commented Dec 20, 2022

First steps:
Installed two emoji font packages (noto-fonts-emoji and ttf-joypixels)

Result: PhantomJS crashes (SIGFPE) if either of these fonts are installed.

No good.

@BooDoo
Copy link
Owner Author

BooDoo commented Dec 20, 2022

Other approaches:

Intercept emoji and...

  1. Replace via TwiMoji JS package?
  2. Force wrap in an explicit font call?

or ditch svg2png (PhantomJS) with headless Chromium, like I've been meaning to do for years... (see #7)

@BooDoo
Copy link
Owner Author

BooDoo commented Dec 30, 2022

ok well we ditched PhantomJS with df9b202 but this is still an issue based on quick testing.

Twimoji JS package or look into if there is a root cause to address, e.g.:

@BooDoo BooDoo added help wanted Extra attention is needed BRIDGE for the SEND_STATUS submodule and removed svg2png unexpected behavior from svg2png labels Dec 30, 2022
@BooDoo
Copy link
Owner Author

BooDoo commented Dec 30, 2022

Overthinking it.

With the switch to chromium and then installing emoji fonts these all seem to render as intended now.

Noto Color Emoji is preferred, JoyPixels can be manually invoked. Need to update documentation.

@BooDoo BooDoo closed this as completed Dec 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BACKEND BRIDGE for the SEND_STATUS submodule help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant