Broken in Edge

[jonathantneal]

Broken in Edge

[JamesJosephFinn]

I, too, can replicate this issue. We upgraded our Windows machine today to Windows 10, and our svg-sprites are broken in Edge.

We're using v1.0.0 – (I didn't even know this project upgraded to 2.0)

[liamness]

The UA string in the VM is just

Mozilla/5.0 (Windows NT 10.0; Win64; x64)

Which means the regex test will fail, it looks for the string Edge/12 which obviously it does not find. 2.0.0 does work in a real version of Windows 10 with Edge, I guess the VM build is just out of date.

[liamness]

<use> is in the latest preview now:

https://wpdev.uservoice.com/forums/257854-microsoft-edge-developer/suggestions/6263916-svg-external-content

[JamesJosephFinn]

Upgrading to 2.0 worked for us. Based on our tests, this issue can probably be closed.

[bdelaney]

hmm, insider preview win10, build 10532 and Edge does not show my SVG sprites. I'm using latest build of SVGforeverybody, 2.0 and doing images with <use>
Will download new insider preview today, build 10547, and see if it works now.
...
Aha! New version of Edge, 21.10547.0.0 works!

[jdalton]

Edge now supports SVG external content.

[srife]

Insider build 14267.rs1_release.160213.0213
with Edge version 31.14267.1000.0 and EdgeHTML version 14.14267
does not work.

[jonathantneal]

@srife, is does not work as in the polyfill unnecessarily applies or that it does not apply and external content is not working?

[srife]

Good question... I was having a problem with WakaTime, which apparently uses this project to display graphs. The graphs on their website were not displaying properly, so I sent a message to WakaTime. After chatting with Alan at Wakatime he pointed out this page and said he was using this to display the graphs.

I've attached an image of what one of the graphs looks like.

Other than that, I'm kind of clueless. I hope this is enough info to let you sort things out.

[alanhamlett]

Currently we removed svg4everybody since it wasn't working, in case someone is checking the production WakaTime website.

Those labels are svg text elements, so it looks like Edge is partially rendering SVG?

[jonathantneal]

@alanhamlett, I’m still looking for details of the issue. Is svg4everybody causing a problem? Is Edge 13 not rendering an external SVG? If you’ve removed svg4everybody, does this mean you are not using external svgs with <use> or that your site no longer works for older Edge and IE9-11 users?

[alanhamlett]

The svgs are not external, they are added to the dom from JS using c3. After rendered I was executing svg4everybody() but nothing changed in the svg.

Is Edge 13 not rendering an external SVG?

Not sure, because these SVGs were never using external content with <use>. I was mistaken when I thought svg4everybody helps render any svg content not just external.

If you’ve removed svg4everybody, does this mean you are not using external svgs with or that your site no longer works for older Edge and IE9-11 users?

The site works for IE9-11 without svg4everybody because we aren't using external svg elements.

This looks like the wrong use-case for svg4everybody since I'm not using external svg elements. It's just a bug in Edge with rendering normal svg.

[jonathantneal]

👍