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 accessing webkitBackingStorePixelRatio of context on iOS10 #26

Conversation

hinok
Copy link
Contributor

@hinok hinok commented Sep 21, 2016

On iOS 10 webkitBackingStorePixelRatio can be accessed only by using
instance of CanvasRenderingContext2D. Using Object.create(prototype)
in that case doesn't work so the only way to access that property is by
getting context 2d from canvas element.

@hinok hinok mentioned this pull request Sep 21, 2016
Copy link
Owner

@jondavidjohn jondavidjohn left a comment

Choose a reason for hiding this comment

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

Nice! Had one nit in the .gitignore, mind removing that line?

@@ -1 +1,3 @@
/node_modules
.idea
Copy link
Owner

Choose a reason for hiding this comment

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

Not sure why we're adding .idea?

Copy link
Owner

Choose a reason for hiding this comment

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

Maybe this is something you could add to your system level .gitignore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

.idea is a storage / config folder created by JetBrains' IDEs like WebStorm / PhpStorm. I added it as just in case if someone in future will also be using one of those IDEs.

I will remove it from PR.

On iOS 10 webkitBackingStorePixelRatio can be accessed only by using
instance of CanvasRenderingContext2D. Using Object.create(prototype)
in that case doesn't work so the only way to access that property is by
getting context 2d from canvas element.
@hinok hinok force-pushed the fix/iOS10-webkitBackingStorePixelRatio branch from 7a9cbfd to 7b212d5 Compare September 23, 2016 08:45
@hinok
Copy link
Contributor Author

hinok commented Sep 23, 2016

I removed .idea from .gitignore.

@jondavidjohn jondavidjohn merged commit 34db1af into jondavidjohn:develop Sep 23, 2016
@jondavidjohn
Copy link
Owner

@hinok Thanks! Will get this released soon.

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