This repository has been archived by the owner on Mar 8, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 17
Safari Mask Icon #127
Comments
I don't see any reason why not, but the feature needs to be added to |
ah gotcha, okay. ill wander over there. thanks |
Cool, I'd be happy to review a CR over there.
Colby Fayock <[email protected]> schrieb am Mi., 22. Aug. 2018,
14:38:
… ah gotcha, okay. ill wander over there. thanks
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#127 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AD2zj4MLece1z_bSaWAuLwUngi690x31ks5uTVDagaJpZM4WGsYS>
.
|
seems like there's a lot of back and forth already about supporting SVG generally. i was going to try to dig a little first, but is the SVG support that you provide from favicons or are you adding that as an additional layer? sounded like supporting SVG was the only thing holding it back |
Unless I misunderstood what you are saying, SVG is already supported by
`favicons`, that is the base icon can be a SVG.
Colby Fayock <[email protected]> schrieb am Mi., 22. Aug. 2018,
14:50:
… seems like there's a lot of back and forth already about supporting SVG
generally. i was going to try to dig a little first, but is the SVG support
that you provide from favicons or are you adding that as an additional
layer? sounded like supporting SVG was the only thing holding it back
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#127 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AD2zj1kc8ZMGB1P4P8pJ3L32jWi7unEvks5uTVOBgaJpZM4WGsYS>
.
|
thats what I was thinking, since you provide the support here, but wonder if it's a different use case. i'll leave a note and see whats up over there Commented on the previous thread itgalaxy/favicons#126 Thanks |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Hey! Any intent of supporting Safari mask icons? Happy to put together a PR :)
The text was updated successfully, but these errors were encountered: