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 gecko engine detection for Firefox 47beta+ #2121

Closed
wants to merge 1 commit into from
Closed

Fix gecko engine detection for Firefox 47beta+ #2121

wants to merge 1 commit into from

Conversation

level420
Copy link
Member

@level420 level420 commented May 2, 2016

To detect the gecko rendering window.navigator.mozApps was used, which was (obviously) removed in Firefox47beta. But there still exists the firefox specific mozContacts API which we additionally use now for detection

To detect the gecko rendering window.navigator.mozApps was used, which was (obviously) removed in Firefox47beta. But there still exists the firefox specific mozContacts API which we additionally use now for detection
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 62.656% when pulling aa2c127 on level420:level420-gecko-engine-ff47 into 641c950 on qooxdoo:master.

@oetiker
Copy link
Member

oetiker commented May 2, 2016

is there any documentation on mozContacts?

@level420
Copy link
Member Author

level420 commented May 2, 2016

@level420
Copy link
Member Author

level420 commented May 2, 2016

But wait with the merge. I've found the following article:

https://www.fxsitecompat.com/en-CA/docs/2016/firefox-os-apis-have-been-hidden-from-web-content/

@level420
Copy link
Member Author

level420 commented May 2, 2016

OK. It seems that navigator.mozContacts will desappear in Firefox 48 :-(

https://developer.mozilla.org/en-US/Firefox/Releases/48#Others

Closing, as it is just a temporary fix until FF48 is released.

@level420 level420 closed this May 2, 2016
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.

3 participants