Daryll Doyle

@jeffpaul I wonder if a more performant approach here may be to re-work the `svg_dimensions()` call to store the data in attachment meta the first time it's called. Then on...

Thanks for the info @jeremymoore. I think this could be a great enhancement moving forward, especially taking CLS into account now. @jeffpaul this could work by wrapping [lines 471-476](https://github.com/10up/safe-svg/blob/develop/safe-svg.php#L471-L476) in...

Thanks for the report, I'll try and get my hands on a Win10 machine to test this out and then I'll update the issue!

Is it possible to get some test cases added for this please?

Hi @xerc any chance you can expand on what you're trying to solve here a bit please?

Thanks @derryberni, it currently doesn't run on 5.6. @ohader I remember that 5.6 support was dropped, but I can't remember why. That said, I'd love your thoughts on trying to...

I like this idea but am slightly concerned with setting `minifyXML` and `removeXMLTag` to null defaults. My concern is that it's a potential BC break and could cause users to...

Interesting, so I guess you're looking for a way to filter the `isHrefSafeValue()` method? I think that could be a good addition, my concern was always that people could allow...

@ohader you're correct. The concern here was that any of the following tags will cause the HTML parser to break out of foreign content and back into HTML. > A...

The `enable-background` attribute [has been deprecated](https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute/enable-background) from the SVG specification for some time. It only seems to be included in Illustrator exports now for some legacy reason. If it's not...