brunob/leaflet.fullscreen

isFullScreen() checks wrong property when no prefix used

prettyv opened this issue · 1 comments

Under current versions of Firefox Developer Edition document.exitFullscreen is defined, so the unprefixed case is hit in fullscreenApi.isFullScreen().

The fullscreen property is called document.fullscreen (all lowercase) as specified here, while currently document.fullScreen is used (https://github.com/brunob/leaflet.fullscreen/blob/master/Control.FullScreen.js#L158), which breaks the check.

@prettyv thx for the hint, feel free to provide a pull request.