Closed Equalizer5118 closed 1 year ago
Alright, removed old ico and added svg instead. What do you think?
Not sure where you got that SVG from, the dynamic SVG is https://github.com/StrataSource/strata-branding/blob/main/i/favicon.svg
Sorry if hard to find, not really sure what's going on with the Strata branding tool tbh. But that one I linked should be used for all favicons.
You should still include an ICO favicon though, for browsers that don't support SVGs.
And you'll need to actually update the HTML to add it. Maybe worth pinging me on someone else in Discord if you're not familiar with HTML
Alright, I'll do that. I'm still learning HTML and GitHub, so I apologize for my inability to do this efficiently.
Looks okay, though I don't have a local setup to test this. JJ can you check it out, or just squash (def worth squashing, commit history is gross) it if you're happy?
I tested this locally and everything checks out on my end
Updated name usage to strata