Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Change default favicon size or allow to define favicon size. #614

Open
kkujawinski opened this issue Nov 12, 2024 · 1 comment
Open

Change default favicon size or allow to define favicon size. #614

kkujawinski opened this issue Nov 12, 2024 · 1 comment

Comments

@kkujawinski
Copy link

kkujawinski commented Nov 12, 2024

💬

Favicons are generated with a fixed size of 16px, which may not look optimal on Retina-like displays.

Is it possible to change the default favicon size? My recommendation is 32px.

For reference:

const int kFaviconSize = 16;

Alternatively, this value could be made configurable via the configuration file.

@kthguru
Copy link

kthguru commented Nov 19, 2024

+1
If the kFaviconSize value is 16 and the icon size is larger than 16, the generated icon will have a gray border. It is a small but significant effect on the quality of the icon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants