Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Docs: add favicon #1698

Merged
merged 8 commits into from
May 2, 2023
Merged

Docs: add favicon #1698

merged 8 commits into from
May 2, 2023

Conversation

WolfGangsta
Copy link
Contributor

I like to set up a browser window for just my docs, all in pinned tabs on Chrome (i.e. just the icon showing). Since Pino didn't have one yet, I decided to see if I couldn't set one up myself.

Referencing the existing pino-tree.png is possible, but because the image isn't square, it looked squashed. I headed over to realfavicongenerator.net and put pino-tree.png through it to get some more standardized favicons.

These 3 files were generated (along with a lot of other special-case ones that I removed since I couldn't easily test them). The website also provided some code to add in, which only needed one tweak--the removal of a / before each href--to make GitHub pages support it.

How do they look? (For smaller icon sizes, there's also the option of using bilinear or some other scaling technique, which looks more pixelated. I went with the default, just because it was the default.)

Generated with realfavicongenerator.net
Code generated from realfavicongenerator.net
This isn't a PWA, so doesn't make sense to have a manifest right?
See last commit, don't think it's necessary.
So it works with GitHub pages
I don't really have a way to test these so I'll stick to the basics
Copy link
Member

@jsumners jsumners left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@mcollina mcollina merged commit 941f908 into pinojs:master May 2, 2023
Copy link

github-actions bot commented May 2, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 2, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants