search

sindresorhus / screenfull

Simple wrapper for cross-browser usage of the JavaScript Fullscreen API
https://sindresorhus.com/screenfull
MIT License
7.08k stars 698 forks source link

isFullScreen tweak #11

Closed miketaylr closed 12 years ago

miketaylr commented 12 years ago

Currently isFullScreen (https://github.com/sindresorhus/screenfull.js/blob/gh-pages/src/screenfull.js#L54) checks for the old document.fullscreen property--but that's not in the latest version of the spec (and not implemented in Opera Next builds).

Maybe add a || fn.fullscreenElement !== null or something afterwards?

miketaylr commented 12 years ago

Actually, just || fn.fullscreenElement probably makes more sense.

sindresorhus commented 12 years ago

Thanks for the report. Fixed in 1.0.2