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

Download chart as PNG is showing the wrong font configuration for mobile #3571

Open
paarriagadap opened this issue May 3, 2024 · 2 comments

Comments

@paarriagadap
Copy link

paarriagadap commented May 3, 2024

Description

When downloading a chart as png, the file shows the wrong fonts. This is only happening on mobile.

Expected behaviour

The png file should look like the the desktop download (see screenshots)

Steps to reproduce

Steps to reproduce the behavior:

  1. Go to this URL on mobile.
  2. Click on "Download"
  3. Click on "Image (PNG)"
  4. Open the downloaded picture.
  5. Do the same in desktop and compare both files.

Screenshots

This is how it looks in mobile:
share-of-population-in-extreme-poverty (9)

This is how it should look (desktop download):
share-of-population-in-extreme-poverty (7)

Environment

For desktop, provide:

  • OS: macOS Sonoma 14.4.1
  • Browser: Chrome
  • Version: 124.0.6367.119 (Official Build) (arm64)

For smartphone, instead provide:

  • Device: Google Pixel 7a
  • OS: Android 14
  • Browser: Chrome
  • Version: 124.0.6367.113

Additional context

@paarriagadap paarriagadap changed the title Download chart as png is showing the wrong font configuration for mobile Download chart as PNG is showing the wrong font configuration for mobile May 3, 2024
@danyx23
Copy link
Contributor

danyx23 commented May 22, 2024

To discuss with @marcelgerber in the next issue triage: we probably don't want to invest time in fixing this in some mobile phone cases - but we could consider moving this code from the client to always use our CF workers. TBD

@marcelgerber
Copy link
Member

Sadly, there's an issue where for some memory-intensive charts, the CF workers will fail with OOM (see #3723) and a thumbnail can thus not be generated.

This makes it so that I'm still a bit hesitant to use dynamic thumbnails in this place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants