circle-flags icon indicating copy to clipboard operation
circle-flags copied to clipboard

Safari blurry icons

Open kostyukdg opened this issue 4 years ago • 5 comments

Hi,

Your library is awesome. It works perfectly on any device except for safari(iOS and desktop). The flags are blurry.

device screenshot

It happens when you load an image from url <img src="https://hatscripts.github.io/circle-flags/flags/br.svg" width="48">

I found solutions but looks tricky <object type="image/svg+xml" data={path} />

It only affects icons from this library. That's why I guess it has some problems with svg structure.

Thanks again for your work!

kostyukdg avatar Sep 17 '21 10:09 kostyukdg

Hi @kostyukdg, I'm glad you like the library. Could you please elaborate or provide more screenshots? Unless I am missing something, the screenshot you provided doesn't appear to show obviously blurry flag icons. Thanks!

HatScripts avatar May 04 '22 10:05 HatScripts

I think he may mean the borders around the icons:

blurry-icons

Maybe Safari is not able to properly render the alpha masks (#11/#18)? I wonder if using <clipPath> instead of <mask> might work better. After all, we only need a binary mask (which is what clipPath provides), not a gradual transparency as mask allows (and we're not using, since the mask path we're using is filled with solid white).

In #18, @climech said that

<mask> was chosen over <clipPath> because the former results in slightly smaller file sizes (exactly 1 byte smaller 🙃).

So if <clipPath> is better supported by renderers (to be determined!), maybe the extra 1 byte would be an acceptable workaround.

waldyrious avatar May 04 '22 13:05 waldyrious

This should be fixed now: https://github.com/HatScripts/circle-flags/releases/tag/v2.5.0

Replace mask with border-radius across all flags:

<svg ...><mask id="a"><circle cx="256" cy="256" r="256" fill="#fff"/></mask><g mask="url(#a)">...</g></svg>

...becomes:

<svg ... style="border-radius:50%">...</svg>

HatScripts avatar May 14 '22 05:05 HatScripts

Nice, @HatScripts! Out of curiosity, why did you go with border-radius instead of clipPath? Smaller files?

Edit: either way, it's nice that it was done via CSS, since now the clipping of the flags to different shapes can be done without tweaking the SVG — see #51!

waldyrious avatar May 14 '22 18:05 waldyrious

Cross-referencing #52 and #53 for future reference. It seems like it might be a good idea to pick (and document) the list of platforms/software we want to target and guarantee proper rendering.

In any case, I'd say that, assuming we want to support the platforms where border-radius doesn't work, it's preferable to have blurry borders in Safari (which might even be solved with using a <clipPath> instead of <mask>), than non-round icons at all elsewhere.

waldyrious avatar May 19 '22 13:05 waldyrious