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 high density display support #459

Closed
wants to merge 4 commits into from
Closed

Conversation

SimonBackx
Copy link

For now, when you specify a size of 320 pixels, it will only generate an image with a size of 320px. But most smartphones have higher density displays, so we also need to add 640px (x2) and 960px (x3) to the srcset attribute (not in the sizes attribute). Adding those sizes to the currently available 'sizes' options, is not a possibility since that will also include it in the sizes attribute (which is not allowed, since that would generate incorrect media conditions)

@RobbieTheWagner
Copy link

Any updates on this? We really need a way to support 2x and 3x in this package!

@jfbloom22
Copy link

I was well on my way integrating this plugin, noticed the quality issue on my 4k 24" monitor and am now removing it. Will be back when high density is supported.

@pi0 pi0 added enhancement New feature or request pending labels Jun 20, 2022
@pi0 pi0 added the v0 label Jul 7, 2022
@SimonBackx SimonBackx closed this Sep 7, 2022
@KaragiannidesAgapios
Copy link

Any updates on this?

@northamerican
Copy link

why was this closed? we still seem to be missing hidpi display support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pending v0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants