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 WebGL image layer rendering on retina displays #3448

Merged
merged 1 commit into from
Apr 13, 2015
Merged

Fix WebGL image layer rendering on retina displays #3448

merged 1 commit into from
Apr 13, 2015

Conversation

elemoine
Copy link
Member

@elemoine elemoine commented Apr 1, 2015

This commit takes the device pixel ratio into account when calculating the matrix used to apply the image to the output canvas.

Fixes #3405.

While working on this I've realized about other pixel ratio-related issues:

  • « Layer hit detection » (a.k.a. forEachLayerAtPixel) doesn't work on hidpi devices, with neither the Canvas renderer nor the WebGL renderer.
  • WebGL vector rendering doesn't work on hidpi devices.

I'll create tickets for these issues.

@tsauerwein
Copy link
Member

LGTM

@bartvde bartvde added this to the v3.5.0 milestone Apr 4, 2015
@fredj
Copy link
Member

fredj commented Apr 7, 2015

LGTM

This commit takes the device pixel ratio into account when calculating the matrix used to apply the image to the output canvas.
@elemoine
Copy link
Member Author

elemoine commented Apr 7, 2015

Thanks for the reviews. I just rebased the branch onto the current master. Will merge if Travis build passes.

@fredj
Copy link
Member

fredj commented Apr 13, 2015

ping @elemoine

elemoine pushed a commit that referenced this pull request Apr 13, 2015
Fix WebGL image layer rendering on retina displays
@elemoine elemoine merged commit e636119 into openlayers:master Apr 13, 2015
@elemoine elemoine deleted the webgl-image-retina branch April 13, 2015 09:16
@Xyrus2000
Copy link

Thanks for the fix. Much appreciated. :)

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.

ImageWMS WebGL bug on Retina displays (Macbook Pro)
5 participants