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

Add option to specify image quality #24

Open
s2b opened this issue Feb 19, 2018 · 4 comments
Open

Add option to specify image quality #24

s2b opened this issue Feb 19, 2018 · 4 comments
Assignees

Comments

@s2b
Copy link
Collaborator

s2b commented Feb 19, 2018

It should be possible to specify/overwrite the image quality option. Ideally, it should be possible to specify multiple qualities, e. g. to generate lower-quality for high dpi images.

@s2b s2b self-assigned this Feb 19, 2018
@DanielRuf
Copy link

This is a must have. It seems by default the image quality is lower than the quality of images generated by f:image/media which is not good. (Re)processing should not manipulate / change the quality by default.

At least the generated images seem to be not as crisp as the normal images and look a bit blurry when you look closely at them.

@s2b
Copy link
Collaborator Author

s2b commented Mar 9, 2018

I'm pretty sure that the image quality is the same because the extension uses the core api for image processing just as the core viewhelpers do. The processing is applied to the original image, so there is no reprocessing which could have added up to bad image quality.

However, responsive images can seem blurrier in the browser because they almost almost always will be scaled by the browsers, which can lead to some quality loss.

@DanielRuf
Copy link

Well, even in fullscreen there is some quality loss.

@s2b
Copy link
Collaborator Author

s2b commented Mar 9, 2018

Ok, I'll have to test this...

@s2b s2b added next and removed backlog labels Mar 15, 2018
@s2b s2b removed the next label Jan 23, 2020
@s2b s2b added the backlog label Feb 19, 2021
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