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

feat: responsive avatar image size #1077

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

@CaiJimmy
Copy link
Owner

I think the current setting is good enough: the maximum size of the avatar is 150x150. The theme generates a single image of 300x300, which covers 1x and 2x. This PR will generate 3 images instead.

@ElastiCourse
Copy link

It would be really nice to use plain SVG as site logo. The resize command ran against Hugo pipelines is hard coded and doesn't adhere to the disabled image processing in Hugo config.

@xiaoxiyao
Copy link
Contributor Author

This is for loading smaller images on screens with low pixel ratios to improve loading speed.
And, the maximum size of the avatar is 120x120

--sidebar-avatar-size: 120px;

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.

3 participants