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

Fix jpeg quality option for jpeg files #2016

Merged
merged 3 commits into from
Feb 11, 2021

Conversation

kulturbande
Copy link
Contributor

@kulturbande kulturbande commented Feb 4, 2021

What is this pull request for?

The quality option was previously ignored, if the image was a JPEG. Now the quality configuration will be added, if the file isn't a JPEG or the quality - option was set.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@tvdeyen
Copy link
Member

tvdeyen commented Feb 5, 2021

@kulturbande Thanks. This is really great 👏🏻

The latest main branch contains a fix for the CI. Do you mind rebasing?

@tvdeyen
Copy link
Member

tvdeyen commented Feb 8, 2021

@kulturbande please rebase your branch with latest main branch and do a force push to remove the commits from the history. Thanks for the contribution

@tvdeyen
Copy link
Member

tvdeyen commented Feb 8, 2021

@kulturbande also please use the PR description as commit description, since it's usefull information that we want to have in the commit history.

@tvdeyen tvdeyen added this to the 5.2 milestone Feb 10, 2021
The quality option was previously ignored if the image was a JPEG.
Now the quality configuration will be added if the file isn't a
JPEG or the quality option was set.
@tvdeyen tvdeyen changed the title fix jpeg quality option for jpeg files Fix jpeg quality option for jpeg files Feb 10, 2021
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

@kulturbande thanks. I added specs and rebased with latest main branch.

@tvdeyen tvdeyen merged commit 9c44beb into AlchemyCMS:main Feb 11, 2021
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.

2 participants