Fix: Default seek, event.target, import await, forward scrubbing, fullscreen inview#311
Merged
oliverfoster merged 3 commits intoissue/298from Jan 13, 2025
Merged
Fix: Default seek, event.target, import await, forward scrubbing, fullscreen inview#311oliverfoster merged 3 commits intoissue/298from
oliverfoster merged 3 commits intoissue/298from
Conversation
oliverfoster
changed the title
joe-allen-89
pushed a commit
that referenced
this pull request
Jul 28, 2025
* Remove memory leak in Internet Explorer workaround, remove mep.less * Removed 'killContextMenuTimer override to remove global delete' since it no longer exists * Update base mediaelement JS and Less files * Remove Flash player * Temporarily remove overrides, add SVG asset and legacy Less (TBD if we'll need it) * Update pluginType to rendererName, comment out event listeners for testing purposes * Config: use legacy prefix, set icon asset location * Added fix from mediaelement/mediaelement#2969 * Add readme to libraries * Readme update * Switch to new namespace mejs__ * Fix container height * Remove .me-plugin references * Update Less, remove bug fix for CC button language selector still appearing when toggleCaptionsButtonWhenOnlyOne: true * Update onPlayerReady/success callback with correct new params, change startLanguage to autoplayCaptionLanguage, move startVolume, partially fix mediaCaptionsChange * Add documentation around onPlayerReady / success callback * Documentation update * Replace method calls to incorrect object * Fix remove * Fix setupPlayPauseToggle() and related methods * Remove mediaelementplayer-legacy.less and will re-add if needed * Remove unused assets and styles * Remove jumpforward/skipback assets/styles * Remove Vimeo scripts and styles * Fullscreen sizing fix * Fix onCaptionsChanged * Minor edits/comments * Remove buildfullscreen override, update pluginType to rendererName * Add loadPlugins, speed plugin * Add jumpforward and skipback features, update loadPlugins() to display error messages * Fix plugin icons, add readme * Remove commented code * Button icon fixes * Fix poster size when iPhoneUseNativeControls: true * Fix for #239 * Fix for firefox fullscreen api * #250 Allow use of aria-pressed on closed captions button * Remove fullscreen fix for Chrome version <17. Apply PR: mediaelement/mediaelement#2971 * Comments * Fix missing bracket * Force the default language so that the aria-label can be localised from Adapt * Use dynamic import for mejs plugins, remove loadPlugins() * Remove promise from plugin import * Remove bug Fix for touch devices to override standard hover event * Remove Bug fix for missing control icons on iOS touch devices * Remove side padding on time elements * Remove bug fix due to fixed pixel size preventing iOS from changing orientation without scaling * Update mediaelement-fullscreen-hook.js to ES6 syntax * Update hasTouch to use device.touch instead of relying on mejs * Move addMejsButtonClass() * Comment out setupPlayPauseToggle() functionality as it seems to just break playing on touch devices and may no longer be needed * Rework plugin loading * Move setting up model options to setupModelOptions() * Move fullscreen hook to main overrides file * fix client side library loading * Reversion * Disable keyboard actions * Enable keyboard but still remove default keyActions * Remove unused code * Remove libraries/mediaelement-and-player from wait queue * Update readme and example.json * Remove unnecessary vendor prefixes from fullscreen hook * Update readme with notes on vimeo/youtube * Fix: Default seek, event.target, import await, forward scrubbing, fullscreen inview (#311) * Move plugin Less files into main less directory to prevent issues when using build includes[] * Restore executable permissions for adapt-contrib-media.js * Restore executable permissions for README.md * Update package description to remove Flash reference * Remove transcript.pdf * Rework plugin overrides, add plugin section to readme * Add source ref for mediaelement-and-player.js * Update mejs__container background to transparent * Rework sources check * YouTube/Vimeo check (WIP) * Update MediaElement.js from 7.0.5 to 7.0.7, readme updates for markdownlint * Check for YouTube or Vimeo URLs * Add default landscape width and height to template, do not set width and height when using iPhoneUseNativeControls * Remove part of PR #320 to fix backwards scrubbing bug * Readme: link to specific tagged version of mediaelement-and-player.js * Add shouldSetSize to determine if width / height should be set as a <video> attribute * Remove unnecessary parentheses * Fix Skip to transcript button visibility * Bump FW version to 5.46.10 for .visually-hidden-focusable usage * Fix FW version --------- Co-authored-by: Oliver Foster <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
requires adaptlearning/adapt-contrib-core#612
fixes #304
Fix
import()returns a promisetimeupdatebeforeseekingwhich prevents the seek blocking behaviourdefaultSeekBackwardIntervalanddefaultSeekForwardIntervalneed to be converted to functions from strings at runtimeonMediaElementTimeUpdateandonMediaElementSeekingdo not haveevent.targeton windows chrome useevent.currentTargetinstead