diff --git a/js/h5p-resizer.js b/js/h5p-resizer.js index 228cc72..772cafd 100644 --- a/js/h5p-resizer.js +++ b/js/h5p-resizer.js @@ -19,7 +19,6 @@ actionHandlers.hello = function (iframe, data, respond) { // Make iframe responsive iframe.style.width = '100%'; - iframe.contentDocument.body.style.height = 'auto'; // Tell iframe that it needs to resize when our window resizes var resize = function (event) { @@ -47,7 +46,14 @@ * @param {Function} respond Send a response to the iframe */ actionHandlers.prepareResize = function (iframe, data, respond) { - respond('resizePrepared'); + // Do not resize unless page and scrolling differs + if (iframe.clientHeight !== data.scrollHeight || + data.scrollHeight !== data.clientHeight) { + + // Reset iframe height, in case content has shrinked. + iframe.style.height = data.clientHeight + 'px'; + respond('resizePrepared'); + } }; /** @@ -59,16 +65,8 @@ * @param {Function} respond Send a response to the iframe */ actionHandlers.resize = function (iframe, data, respond) { - if (iframe.clientHeight === iframe.contentDocument.body.scrollHeight && - iframe.contentDocument.body.scrollHeight === iframe.contentWindow.document.body.clientHeight) { - return; // Do not resize unless page and scrolling differs - } - - // Reset iframe height, in case content has shrinked. - iframe.style.height = iframe.contentWindow.document.body.clientHeight + 'px'; - // Resize iframe so all content is visible. Use scrollHeight to make sure we get everything - iframe.style.height = iframe.contentDocument.body.scrollHeight + 'px'; + iframe.style.height = data.scrollHeight + 'px'; }; /** diff --git a/js/h5p-x-api.js b/js/h5p-x-api.js index 6becdad..1cf8b74 100644 --- a/js/h5p-x-api.js +++ b/js/h5p-x-api.js @@ -95,7 +95,8 @@ H5P.EventDispatcher.prototype.triggerXAPIScored = function (score, maxScore, ver H5P.EventDispatcher.prototype.setActivityStarted = function() { if (this.activityStartTime === undefined) { // Don't trigger xAPI events in the editor - if (H5PIntegration.contents !== undefined && + if (this.contentId !== undefined && + H5PIntegration.contents !== undefined && H5PIntegration.contents['cid-' + this.contentId] !== undefined) { this.triggerXAPI('attempted'); } diff --git a/js/h5p.js b/js/h5p.js index 24d9846..934d8f6 100644 --- a/js/h5p.js +++ b/js/h5p.js @@ -279,6 +279,9 @@ H5P.init = function (target) { // Initial setup/handshake is done parentIsFriendly = true; + // Make iframe responsive + document.body.style.height = 'auto'; + // Hide scrollbars for correct size document.body.style.overflow = 'hidden'; @@ -289,7 +292,7 @@ H5P.init = function (target) { // When resize has been prepared tell parent window to resize H5P.communicator.on('resizePrepared', function (data) { H5P.communicator.send('resize', { - height: document.body.scrollHeight + scrollHeight: document.body.scrollHeight }); }); @@ -307,7 +310,10 @@ H5P.init = function (target) { resizeDelay = setTimeout(function () { // Only resize if the iframe can be resized if (parentIsFriendly) { - H5P.communicator.send('prepareResize'); + H5P.communicator.send('prepareResize', { + scrollHeight: document.body.scrollHeight, + clientHeight: document.body.clientHeight + }); } else { H5P.communicator.send('hello'); @@ -1947,20 +1953,35 @@ H5P.createTitle = function (rawTitle, maxLength) { } if (H5PIntegration.saveFreq !== false) { + // When was the last state stored + var lastStoredOn = 0; // Store the current state of the H5P when leaving the page. - H5P.$window.on('beforeunload', function () { - for (var i = 0; i < H5P.instances.length; i++) { - var instance = H5P.instances[i]; - if (instance.getCurrentState instanceof Function || - typeof instance.getCurrentState === 'function') { - var state = instance.getCurrentState(); - if (state !== undefined) { - // Async is not used to prevent the request from being cancelled. - H5P.setUserData(instance.contentId, 'state', state, {deleteOnChange: true, async: false}); + var storeCurrentState = function () { + // Make sure at least 250 ms has passed since last save + var currentTime = new Date().getTime(); + if (currentTime - lastStoredOn > 250) { + lastStoredOn = currentTime; + for (var i = 0; i < H5P.instances.length; i++) { + var instance = H5P.instances[i]; + if (instance.getCurrentState instanceof Function || + typeof instance.getCurrentState === 'function') { + var state = instance.getCurrentState(); + if (state !== undefined) { + // Async is not used to prevent the request from being cancelled. + H5P.setUserData(instance.contentId, 'state', state, {deleteOnChange: true, async: false}); + } } } } + }; + // iPad does not support beforeunload, therefore using unload + H5P.$window.one('beforeunload unload', function () { + // Only want to do this once + H5P.$window.off('pagehide beforeunload unload'); + storeCurrentState(); }); + // pagehide is used on iPad when tabs are switched + H5P.$window.on('pagehide', storeCurrentState); } /**