chore: svg border

[yashmehrotra]

cc: @jamesericdavidson

[netlify]

✅ Deploy Preview for gokarna-hugo ready!

Name	Link
🔨 Latest commit	93eb62c
🔍 Latest deploy log	https://app.netlify.com/sites/gokarna-hugo/deploys/679a4c137299f60008be7328
😎 Deploy Preview	https://deploy-preview-319--gokarna-hugo.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[yashmehrotra]

vs

[526avijitgupta]

@yashmehrotra The mail icon is not visible after the changes in this PR. Check deploy preview:

Also, the border lines look blurry for some reason:

[jamesericdavidson]

@ 93eb62c Tested locally - no obvious regressions.

@yashmehrotra How are you testing the Bluesky icon? Can you provide a diff patch, please?

[yashmehrotra]

@jamesericdavidson Downloaded the bluesky icon from simpleicons, and config.toml looks like:

    [[menu.main]]
    identifier = "bluesky"
    pre = "<img class='svg-inject' src='/https/github.com/icons/bluesky.svg' />"
    url = "https://bsky.app/profile/yashmehrotra.com"

[526avijitgupta]

Lgtm 🚀

[jamesericdavidson]

LGTM

[jamesericdavidson]

@yashmehrotra @526avijitgupta Should we support svg-inject and user-specified SVG icon sets in the advanced docs?

[yashmehrotra]

@jamesericdavidson That is already mentioned in the docs

https://gokarna-hugo.netlify.app/posts/theme-documentation-advanced/#icons-in-header

[526avijitgupta]

Should we merge this? @yashmehrotra @jamesericdavidson

[jamesericdavidson]

Should we merge this? @yashmehrotra @jamesericdavidson

Yes 👍 @526avijitgupta