Skip to content
This repository has been archived by the owner on Dec 20, 2021. It is now read-only.

Make image quality configurable for posts single headers #109

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

fmueller
Copy link

@fmueller fmueller commented Jun 4, 2021

I use some high-quality photos for blog posts and found the quality of 100 for downsized images a bit too much. I don't see a difference with 90 quality and save a lot of bandwidth.

This PR makes this configurable for 'hero' and 'full' themes. Default is 90.

If this does not match with your design goals for this template, please close this or use it as an inspiration.

Thank you for making this open source. :)

@uPagge
Copy link
Owner

uPagge commented Jun 4, 2021

It's a great idea to make this parameter configurable.

@uPagge uPagge self-assigned this Jun 4, 2021
@uPagge uPagge added the feature label Jun 4, 2021
@fmueller
Copy link
Author

fmueller commented Jun 5, 2021

Uh damn it, sorry all the other commits from my custom fork slipped in here. This was unintended. I don't have the permissions to clean it up.

Please feel free to close this and do the changes structured.

Sorry for the inconvenience.

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

Successfully merging this pull request may close these issues.

2 participants