Skip to content

Update favicon to higher-resolution version#1366

Merged
Socialhappiness merged 1 commit intomasterfrom
favicon
Mar 19, 2026
Merged

Update favicon to higher-resolution version#1366
Socialhappiness merged 1 commit intomasterfrom
favicon

Conversation

@jacobgkau
Copy link
Member

The support docs website currently uses a lower-resolution favicon than our main website and the Tech Docs website. This updates to the PNG the Tech Docs one uses, optimized with oxipng (only 1.68 KiB more than the .ico it's replacing).

Comparison of the favicons before this fix (main, tech docs, support docs):

Screenshot_2026-03-13_13-53-31

@jacobgkau jacobgkau requested review from a team March 13, 2026 20:11
@jacobgkau
Copy link
Member Author

I figure sticking with PNG instead of a newer-gen format might be safer for this specifically because it's the favicon, but open to alternative suggestions if web has them.

s76-odoo
s76-odoo previously approved these changes Mar 13, 2026
Copy link

@s76-odoo s76-odoo left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@erinxocon erinxocon left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Socialhappiness Socialhappiness 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

@Socialhappiness Socialhappiness merged commit b9c5974 into master Mar 19, 2026
3 checks passed
@Socialhappiness Socialhappiness deleted the favicon branch March 19, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants