cwa-documentation icon indicating copy to clipboard operation
cwa-documentation copied to clipboard

Minor: web-safe fonts in svg figures

Open noleti opened this issue 4 years ago • 3 comments

This one is minor, wanted to provide feedback nevertheless: figures are (at least in github) embedded as SVG directly. Fonts that are currently used include "Helvetica Neue". For systems without those fonts installed, the rendering in the browser (and other applications) will be impacted.

Example rendering on my machine (vanilla Ubuntu 18.04) test

Where to find the issue

Any of the svg images (e.g., https://github.com/corona-warn-app/cwa-documentation/raw/master/images/solution_architecture/figure_1.svg?sanitize=true)

Describe the issue

  • see above, svg is rendered with incorrect spacing due to localy missing font.

Suggested change

  • It seems that normally the font would be provided by the hosting page, not sure how this works in github.
  • Embed the images as PNG (in addition to having the svg sources here)
  • Embed the font directly (quick googling suggests https://github.com/BTBurke/svg-embed-font or https://vecta.io/nano).

Internal Tracking ID: EXPOSUREAPP-2964

noleti avatar May 19 '20 12:05 noleti

Thanks for that input. We'll have a look and try to fix that

tkowark avatar May 19 '20 13:05 tkowark

also affecting standard manjaro linux and whonix installs.

possible workaround for the moment: Open the svg in a (firefox) tab and highlight the text in question. The highlighted characters will then at least not overlap as much as before making it a bit more readable.

jn4kr avatar May 19 '20 17:05 jn4kr

I think this, now nearly two year old, issue was never addressed. It makes no sense to leave issues open for two years without any process on them. I suggest to either fix this issue or apply the https://github.com/corona-warn-app/cwa-documentation/labels/wontfix label here and close it.

Ein-Tim avatar Apr 18 '22 12:04 Ein-Tim

Dear community,

this issue became stale. A simple solution would be to offer alternatively PNG images. However, we are closing this issue now; should you consider that this should be fixed, please re-open. Thanks.

Best wishes, DS


Corona-Warn-App Open Source Team

dsarkar avatar Oct 27 '22 11:10 dsarkar