Skip to content

Commit 4925bac

Browse files
committed
iOS issue attempt: changed event listener to listen for fullscreen events on "player" element instead of document. Used a variable to keep track of fullscreen status
1 parent 2255b03 commit 4925bac

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

src/PlayerScripts.js

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,11 @@ export const MAIN_SCRIPT = (
9898
var firstScriptTag = document.getElementsByTagName('script')[0];
9999
firstScriptTag.parentNode.insertBefore(tag, firstScriptTag);
100100
101+
var isFullScreen = false;
102+
function onFullScreenChange() {
103+
isFullScreen = !isFullScreen;
104+
window.ReactNativeWebView.postMessage(JSON.stringify({eventType: 'fullScreenChange', data: isFullScreen}));
105+
}
101106
102107
var player;
103108
function onYouTubeIframeAPIReady() {
@@ -130,10 +135,8 @@ export const MAIN_SCRIPT = (
130135
'onPlaybackRateChange': onPlaybackRateChange,
131136
}
132137
});
133-
}
134138
135-
function onFullScreenChange() {
136-
window.ReactNativeWebView.postMessage(JSON.stringify({eventType: 'fullScreenChange', data: !!document.fullscreenElement}))
139+
document.getElementById('player').addEventListener('webkitfullscreenchange', onFullScreenChange);
137140
}
138141
139142
function onPlayerError(event) {
@@ -156,14 +159,6 @@ export const MAIN_SCRIPT = (
156159
function onPlayerStateChange(event) {
157160
window.ReactNativeWebView.postMessage(JSON.stringify({eventType: 'playerStateChange', data: event.data}))
158161
}
159-
160-
document.addEventListener('fullscreenchange', onFullScreenChange)
161-
162-
document.addEventListener('mozfullscreenchange', onFullScreenChange)
163-
164-
document.addEventListener('msfullscreenchange', onFullScreenChange)
165-
166-
document.addEventListener('webkitfullscreenchange', onFullScreenChange)
167162
</script>
168163
</body>
169164
</html>`;

0 commit comments

Comments
 (0)