Svg-sprite-loader: Mention in README that this is not working in Safari

[vchrm]

Hi, the solution does not work in Safari :-(. The extraction works great, linking to sprite.svg#fragment from background-image CSS attribute is good with Chrome, but in Safari you have no luck.

Mentioning this in README would save me few hours implementing the package into our dev stack. We cannot afford having no icons in Safari.

Would it be a problem to mention this right away? Thank you.

[kisenka]

Hi! Could you please make a PR?

[kisenka]

@vchrm ping

[vchrm]

Yes, but not this week (too much work). Moreover I am still investigating the issue (trying to find some working polyfill or something).

[kisenka]

Check this one https://github.com/Keyamoon/svgxuse

[coderitual]

I have the same issue. Unfortunately svgxuse is not working for this purpose. Any other solution? Maybe this has something to do with the way svg-sprite-loader is creating fragments ids in external file (exposed by use element referenced internally to symbol).

[kisenka]

@vchrm currently it's a planned feature for 4.x (see kisenka/svg-baker#4), but as usual I have no time to implement it :) BTW the only working approach among all browsers is to create classic sprite and move it's background-position. I'll try to find time on next week.

[vchrm]

Do we agree that this is what Safari is missing? https://caniuse.com/#feat=svg-fragment

[kisenka]

@vchrm yes, both, desktop and mobile.

[coderitual]

The simplest solution is to use object instead of img. Not sure if this has some disadvantages.

[josemartos]

Looking forward to hearing from this 👍