Handle external images for SVGs in privacy plugin#7650
Handle external images for SVGs in privacy plugin#7650squidfunk merged 1 commit intosquidfunk:masterfrom
Conversation
cf63558 to
c79ac7c
Compare
|
Thanks for the PR! Could you please provide a minimal reproduction, so we can understand what this PR is trying to solve exactly? Also, if no trouble, could you please include the other cases we already support, so we can estimate whether this is something we're happy to support? As mentioned, always a good idea to talk to us first to omit duplicate work, but I think this is something worth supporting and should go into master, once we know that it aligns with our goals 😅 |
Ah sorry got a little ahead of myself, will provide a little repro project for both! |
|
No worries! At some point in the future we'll have tests, that'll make it easier 😅 |
|
@squidfunk here's a mini repro: 9.5.42-external-svg-images-excluded-in-privacy-plugin.zip You can see with the current version the background is still loaded from the external source, which won't work offline. I just used https://www.w3schools.com/graphics/tryit.asp?filename=trysvg_image2 but I think you'll get the idea 🙂 I'll get to the other issue in a bit! |
|
This one LGTM, thanks for the PR again! |
|
Released as part of 9.5.43. |
Some users (like us) might have quirks around serving SVG images/logos, so we include them from remote locations, e.g. for licensing reasons etc.
Since we can't include them locally but some users would like to have offline builds, this PR adds support for loading remote images from SVGs in the privacy plugin. WDYT? See also some references below 🙇
<image>reference: https://developer.mozilla.org/en-US/docs/Web/SVG/Element/image🛠️ with ❤️ at Siemens