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

pointerlock API feature detect #655

Merged
merged 1 commit into from Aug 20, 2012
Merged

pointerlock API feature detect #655

merged 1 commit into from Aug 20, 2012

Conversation

johnmccutchan
Copy link
Contributor

Hi,

This seemed too easy- let me know if I've missed the hard part.

John

@travisbot
Copy link

This pull request passes (merged 1632ae0 into b18048b).

@@ -0,0 +1,4 @@
// https://developer.mozilla.org/en-US/docs/API/Pointer_Lock_API

Modernizr.addTest('pointerlock',!!Modernizr.prefixed('pointerLockElement', document));
Copy link
Member

Choose a reason for hiding this comment

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

I'd probably have went with

Modernizr.prefixed('requestPointerLock', window)

But it doesnt seem to matter much. Spec likes both. http://dvcs.w3.org/hg/pointerlock/raw-file/default/index.html

@paulirish
Copy link
Member

I'm 👍 as long as no one else has qualms

@ryanseddon
Copy link
Member

It only talks about the Element and Document interface in the link has to be an actual element to do any pointerlock stuff any right? You're essentially requesting pointerlock access to a specific element. So LGTM. :shipit:

paulirish added a commit that referenced this pull request Aug 20, 2012
pointerlock API feature detect
@paulirish paulirish merged commit eeac09d into Modernizr:master Aug 20, 2012
@paulirish
Copy link
Member

merged.

patrickkettner pushed a commit to patrickkettner/Modernizr that referenced this pull request Feb 22, 2015
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.

None yet

4 participants