#39686 closed enhancement (fixed)
Upgrade MediaElement to 4.X
Reported by: | rafa8626 | Owned by: | |
---|---|---|---|
Milestone: | 4.9 | Priority: | high |
Severity: | normal | Version: | 4.7.1 |
Component: | External Libraries | Keywords: | has-dev-note |
Focuses: | accessibility, javascript | Cc: |
Description
The latest version of MediaElement (3.0.2 by the end of the week) has covered many new media types. But there are things that need to be updated on your side to make it work:
- Remove prior files and include new files inside wp-includes/js/mediaelement folder
- On wp-includes/js/mediaelement/wp-mediaelement.js:
- Remove lines related to Silverlight (since it’s no longer supported)
- Change line if ('flash' === mejs.pluginType) to if (mejs.rendererName.match(/flash/gi) !== null)
- Add in the player settings classPrefix: 'mejs-' to support prior class styles, since now uses BEM naming convention.
- Replace wp-includes/js/mediaelement/wp-mediaelement.min.js with changes specified above.
- Modify wp-includes/script-loader.php to update the reference to MediaElementJS changing lines:
- $scripts->add( 'mediaelement', "/wp-includes/js/mediaelement/mediaelement-and-player.min.js", array('jquery'), '3.0.0', 1 );
- $styles->add( 'mediaelement', "/wp-includes/js/mediaelement/mediaelementplayer-legacy.min.css", array(), '3.0.0');
- Translations provided in mejsL10n; refer to https://github.com/johndyer/mediaelement/blob/master/src/js/languages/en.js to see new format and whole list of translations
- It’s worth to expand your Embeds code to use MediaElementJS to support new formats such as HLS, M(PEG)-DASH, YouTube, Vimeo, SoundCloud, Facebook and DailyMotion.
Attachments (10)
Change History (142)
#1
@
8 years ago
- Component changed from Embeds to External Libraries
- Keywords needs-patch added
- Summary changed from Support MediaElementJS latest version to Upgrade MediaElement to 3.0.2
- Type changed from feature request to enhancement
#4
@
8 years ago
- Summary changed from Upgrade MediaElement to 3.0.2 to Upgrade MediaElement to 3.1.0
As an FYI, version 3.1.0 will be available on Feb 2, 2017 so please update to that one, please.
#5
@
8 years ago
- Summary changed from Upgrade MediaElement to 3.1.0 to Upgrade MediaElement to 4.0.3
Version 4.0.3 has been released https://github.com/mediaelement/mediaelement/blob/master/changelog.md
The current version in WordPress (2.22.0) has a lot of bugs.
Do you plan an upgrade for the next WP minor release?
#6
@
8 years ago
- Milestone changed from Awaiting Review to 4.7.5
2.22.0 to 4.0.3 is huge bump. We need to make sure everything still works as expected.
#7
@
8 years ago
Hi, I discovered that the latest iOS and Mac OS version with Safari are not able to play audio files with mediaelement.js player any more. All other operating systems and browsers (Windows, Linux, Mac with all other major browsers) are still working, a quick fix would be perfect ;-)
#8
@
8 years ago
- Summary changed from Upgrade MediaElement to 4.0.3 to Upgrade MediaElement to 4.0.4
#9
@
8 years ago
@Plastikschnitzer Were you using the 4.0.4 version of the 2.22.0 version?
#10
@
8 years ago
Sorry for being unclear here, I was using the mediaelement.js from wp 4.7.3, my developer said its version 2.22.0 of mediaelement.js and he told me this should be updated asap.
So I searched a bit and found this ticked and saw that its already planned but I was not sure if you are aware of the fact that the version which is shipped in Wordpress core 4.7.3 does not work for iOS/Mac OS latest version and Safari – which are a lot of users (and I am wondering that there are no other people writing about it online).
This ticket was mentioned in Slack in #core-media by swissspidy. View the logs.
8 years ago
#12
@
8 years ago
- Summary changed from Upgrade MediaElement to 4.0.4 to Upgrade MediaElement to 4.0.5
This ticket was mentioned in Slack in #core-media by andreiglingeanu. View the logs.
8 years ago
#14
@
8 years ago
- Summary changed from Upgrade MediaElement to 4.0.5 to Upgrade MediaElement to 4.0.6
#15
@
8 years ago
Hi, will this be supported soon ?
The fix to make it work is quite a mess.
Thanks a lot.
https://github.com/mediaelement/mediaelement/blob/master/docs/installation.md#wordpress
#16
@
8 years ago
- Summary changed from Upgrade MediaElement to 4.0.6 to Upgrade MediaElement to 4.0.7
#17
@
8 years ago
- Keywords has-patch added; needs-patch removed
I made a patch (for MediaElement.js 4.0.7) based on the official MediaElement guide, it is attached to the ticket.
https://github.com/mediaelement/mediaelement/blob/master/docs/installation.md#wordpress
#18
@
8 years ago
Looking at mediaElement.js.4.0.7.diff:
- Adding new file types/extensions (
m3u8
andmpd
) to WordPress is not part of updating MediaElement.js and needs its own ticket. Also not sure how "safe" these are. - Same for the changes to
wp_get_audio_extensions()
. They seem to be out of scope here and will need separate ticket if warranted. - Don't think we need to translate the languages names specifically. Instead will need to add the WordPress translations to ME.js.
mejs.rendererName.match(/flash/gi) !== null
is probably better as/flash/i.test( mejs.rendererName )
.
#20
@
8 years ago
Don't think we need to translate the languages names specifically. Instead will need to add the WordPress translations to ME.js.
I'm not sure of it (I just followed their guide), but I think the translations (at least, the JS variables) are required by the MediaElement.js library.
Can I help for this ? I would really appreciate the library upgrade in an upcoming release of WP...
#21
@
8 years ago
- Keywords has-patch added
Here's another shot without modifications to wp_check_filetype(), wp_get_mime_types(), wp_get_ext_types() wp_get_audio_extensions(), wp_get_video_extensions() and wp_audio_shortcode().
#22
@
8 years ago
- Summary changed from Upgrade MediaElement to 4.0.7 to Upgrade MediaElement to 4.X
4.1.0 (2017/05/04) is out, 4.1.1 in progress..
This ticket was mentioned in Slack in #core by jeffpaul. View the logs.
8 years ago
#25
@
8 years ago
- Milestone changed from 4.7.5 to 4.8.1
If the refresh and testing can finish by Friday, May 12th then we can look to bring it back to the 4.8 milestone.
#26
@
8 years ago
Wanted to note here that during testing of the media widgets, a bug in the current version of MediaElement was hit:
https://github.com/xwp/wp-core-media-widgets/issues/130
Currently the bug causes YouTube videos to loop even when the wp_video_shortcode
has loop set to false.
#29
@
8 years ago
Also, I think this issue is on version 2.x if I'm not mistaken. Please correct me if that's true
#30
@
8 years ago
@rafa8626 :
You can see it in action here : http://bit.ly/2p6XH3K - 4.1.0.
BTW, it seems that something (player css styles) does not load.
Could you help me find what ?
#31
follow-up:
↓ 32
@
8 years ago
@grosbouff In the player configuration, use classPrefix: 'mejs-' so we don't break any of the existing themes in WordPress. Try that and I'll check the other issue
#32
in reply to:
↑ 31
@
8 years ago
Replying to rafa8626:
@grosbouff In the player configuration, use classPrefix: 'mejs-' so we don't break any of the existing themes in WordPress. Try that and I'll check the other issue
great, it's working !
#33
@
8 years ago
@grosbouff Great news! Just to confirm: the issue that @timmydcrawford was talking about is also fixed? Or that's a different issue?
#34
@
8 years ago
I don't know about that one. Maybe you should ask in the related ticket : https://github.com/xwp/wp-core-media-widgets/issues/130
#35
follow-up:
↓ 42
@
8 years ago
Sorry for not adding some detailed instructions to my prior report. The issue we are experiencing is related to the current version of mediaelement which in trunk appears to be 2.22.0 - at least that is what is output when I type mejs.version
in the browser console.
The Media Video Widget uses wp_video_shortcode
, which for a YouTube video url, outputs the following in the DOM:
<div style="width:100%;" class="wp-video"><video class="wp-video-shortcode" id="video-1-6" preload="metadata" controls="controls"> <source type="video/youtube" src="https://www.youtube.com/watch?v=b5Qh72h8vkk&_=6" /><a href="https://www.youtube.com/watch?v=b5Qh72h8vkk">https://www.youtube.com/watch?v=b5Qh72h8vkk</a></video> </div>
From there mejs takes over. The corresponding mejs bug is detailed here https://github.com/mediaelement/mediaelement/issues/1851 and was fixed subsequently in https://github.com/instructure/mediaelement/pull/17. The bug fix was released in version 2.23.2 per https://github.com/instructure/mediaelement/blob/master/changelog.md
To see the bug happen, take the HTML above which is output from wp_video_shortcode
, and drop it into a text widget on a site running trunk. Then view the output widget, click play - let it play to the end, and note that the video loops. I attempted to fix by adding a loop=0
to the embed url, which YouTube supports, but the bug is in medialement itself it seems.
Let me know if I can provide any more details
#36
@
8 years ago
@timmydcrawford Does this mean that this issue that is in 2.22.0 version needs to be fixed in order to upgrade to 4.1.0? Or am I missing something? I'd be happy to assist with this, but I'm failing to see how is this gonna help for the upgrade
#38
@
8 years ago
@grosbouff Where did you set _wpmejsSettings? That's the last thing missing in the patch to work globally, since there it needs to include the classPrefix: 'mejs-' configuration, but I can't find it. If you can just add that to the latest patch you submitted this should be good
#39
@
8 years ago
@rafa8626
sorry, I don't understand everything about all that stuff yet.
Can you explain this ?
Where should I put it and using which values ?
I just followed the guide here : https://github.com/mediaelement/mediaelement/blob/master/docs/installation.md#wordpress
and it's working for me that way.
Please be more explicit about this :)
We're almost there !
#40
follow-up:
↓ 41
@
8 years ago
@grosbouff Just add this line on the patch after the translations; I finally found it:
$scripts->add( 'wp-mediaelement', "/wp-includes/js/mediaelement/wp-mediaelement$suffix.js", array('mediaelement'), false, 1 );
$mejs_settings = array(
'pluginPath' => includes_url( 'js/mediaelement/', 'relative' ),
'classPrefix' => 'mejs-', This will be a global element that will help all the themes
);
#41
in reply to:
↑ 40
@
8 years ago
Replying to rafa8626:
@grosbouff Just add this line on the patch after the translations
Ok, it's done in -v3 !
#42
in reply to:
↑ 35
@
8 years ago
@timmydcrawford I submitted an attachment with the files with the fix for YouTube's loop; let me know if you need anything else from me. I'll check if this is not an issue with 4.x and I'll fix it in that version if needed. Just let me know if this will help to upgrade the package soon
#43
@
8 years ago
@timmydcrawford I've checked this on 4.1.0 version and there are not issues at all with this. Just an FYI
#44
@
8 years ago
@rafa8626 thanks for looking into this and verifying it is fixed in 4.1 as well. @westonruter do you think we can just wait for the mejs upgrade to go through, or do we need to pursue patching the existing version, or upgrading on the 2.x branch for the video widget?
#45
@
8 years ago
@rafa8626 @timmydcrawford can the fix be supplied as a patch file please?
I don't have prior experience with upgrading ME.js in core, so I'm not familiar with the best protocol for upgrading vs patching.
#46
@
8 years ago
@westonruter If you can indicate how to create a patch I'll do it. I don't have experience with these type of requests so any hints will be appreciated. Thanks
#47
@
8 years ago
Can I help for this ? I know how to make a diff file but i'm not sure to understand what I have to do...
#48
@
8 years ago
@grosbouff If you can just provide me the command you use for diff I can submit this as a diff file. Or are you using pure git diff command?
#51
@
8 years ago
Hi,
we did some further testing to my case above (current wp-stable release with mediaelement.js in version 2.22.0) with iOS 10 / Safari mobile devices and current stable Mac OS / Safari: Audio is working if the site has not https! If the site is running on https, its broken.
Can we make sure https and the OS/Browser mentioned above is working with this fix?
Someone already described the bug and built a fix inspired by youtubes way to come around the bug here: https://poodll.com/ios-10-and-html5-video-and-html5-audio-on-https-sites/
Best regards,
Plastikschnitzer
#52
@
8 years ago
@Plastikschnitzer Certainly this workaround will be introduced in 4.1.1, but I don't know what's the next step in terms of maintaining the 2.22.0 version or moving forward with 4.1.1
#53
@
8 years ago
@Plastikschnitzer I'll add the fix for 2.22.0 just to ensure that this will give confidence to move forward with 4.1.1
#55
@
8 years ago
@Plastikschnitzer I'll submit both the fixed files and diff to make sure everybody's covered
#56
@
8 years ago
@Plastikschnitzer It's ready you will find the zip file with the fixes; please test and let me know if it worked for you
#57
@
8 years ago
@westonruter The final diff is called mejs-2.22.0-fixes and it includes the fix for YouTube and what @Plastikschnitzer indicated as a bug for iOS. Let me know if you need anything else
#58
follow-up:
↓ 59
@
8 years ago
Thank you very much for your quick fix! I applied your patch and now its working on both iOS / mobile Safari and latest Mac OS / Safari with the website on https.
Thank you very much @rafa8626!
#59
in reply to:
↑ 58
@
8 years ago
Thank you very much for your quick fix! I applied your patch and now its working on both iOS / mobile Safari and latest Mac OS / Safari with the website on https.
Thank you very much @rafa8626!
You are welcome! I'm glad it worked for you. @westonruter Any way I can help you with the MEJS upgrading?
#60
@
8 years ago
Replying to jbpaul17:
If the refresh and testing can finish by Friday, May 12th then we can look to bring it back to the 4.8 milestone.
Ok guys, tomorrow is the deadline :) Is everything ready for it ?
#61
@
8 years ago
@grosbouff Can you create a final diff file in which reflects this on the wp-includes/js/mediaelement/wp-mediaelement.js file, please?
Change
mejs.rendererName.match(/flash/gi) !== null
to
~mejs.rendererName.indexOf('flash')
#63
@
8 years ago
@rafa8626 : I uploaded mediaElement.js.4.1.0-v4.diff.
The ONLY thing that changed since mediaElement.js.4.1.0-v3.diff is what you asked for in #61(https://core.trac.wordpress.org/ticket/39686#comment:61).
(I also did re-minify wp-mediaelement.js > wp-mediaelement.min.js using https://jscompress.com/)
Is everything OK with this and can we tell the WP team ?
This ticket was mentioned in Slack in #core by rafa8626. View the logs.
8 years ago
#65
@
8 years ago
- Focuses accessibility added
Worth noting upgrading is also a matter of improved usability and accessibility. For example, when using the keyboard and tabbing through the player controls, once the last focusable control gets blurred, all the controls fade out:
While this makes sense when playing a video, it happens also when playing an audio file:
Controls get hidden offscreen, they're still focusable and actionable but visually completely hidden. This was resolved in MediaElement version 2.22.1 (WordPress currently uses 2.22.0) released on 2016/08/18, see changelog and see related pull request on GitHub.
Worth noting that the controls don't come back into view when tabbing backwards, it seems to me this happens also on the latest MediaElement version and it would be great if it could be fixed upstream. /cc @rafa8626 will create an issue on GH :)
I'd just like to note that not including the non-minified version in WordPress makes debugging this kind of things a bit hard.
#66
@
8 years ago
@afercia The fix for Accessibility you mentioned above is ready to be released in the next version of MediaElement. Please confirm that it works so we can generate a new patch. @grosbouff Are you up for the task once 4.1.1 is released?
#69
@
8 years ago
@grosbouff 4.1.1 version has been released; however, @afercia pointed some accessibility issues that will be fixed on 4.1.2, that will be released by the end of next week. Also, if you download https://github.com/mediaelement/mediaelement/files/1009558/wp-mediaelement.zip it contains the latest version of the wp-mediaelement files with some minor formatting fixes. So let's create the new patch once 4.1.2 is released. Thanks
#71
@
8 years ago
@swissspidy
#40788 was marked as a duplicate. It looks like the reported issue was not understood correctly and I believe this was marked unfairly as duplicate as it is not a duplicate at all of this thread in here.
Its a new issue that #40788 reports. 1px border because of the mejs scripts wp is using and which will popup the moment the users start using the video widget in wp 4.8 in a sidebar. You can even see it in here at the screenshot @afercia added.
Even if the suggested modifications in this thread are applied that 1px issue will remain as long as wp does not update their MEJS scripts to the latest build 4.1.1 or above. Or adjust the mejs script wp is using according to the suggestion in this thread $40788
#72
@
8 years ago
Even if the suggested modifications in this thread are applied that 1px issue will remain as long as wp does not update their MEJS scripts to the latest build 4.1.1 or above.
@BackuPs I'm sorry if I might have misunderstood you, but that's what this ticket is about: updating MEJS to the latest version. If a planned regular update also resolves that 1px border issue, that's even better.
#73
@
8 years ago
@swissspidy
The milestone for fixing this is set at 4.8.1 whereas the new video widget is released at 4.8 And that has the 1px border issue as it using the old MEJS script. So the question i have: is the milestone gonna be changed to 4.8 to update to newer MEJS scripts? Or do we still have to live with that issue amongst the one reported above in this thread in wp 4.8?
Thanks !
#74
@
8 years ago
- Keywords needs-patch added; has-patch removed
@grosbouff @rafa8626 The patch in mediaElement.js.4.1.0-v4.diff did not apply cleanly so I tried fixing it up as well as make some additional changes needed with the introduction of media widgets.
Note that source files in WP should not be re-minified and submitted as part of patches, since the WP build process takes care of that. The minified files generated as part of a ME.js build are different and should be included.
There is an error about mejs.rendererName
not being defined in wp-mediaelement.js
; this prevents all players from loading. Workaround in 868a0c2.
There is a 404 for mejs-controls.svg
. Added in e613b03.
I am noticing a few issues with the patch, perhaps in part to my refresh of the patch. The YouTube and Vimeo players are not working very well or not at all. I've added some screenshots here: https://github.com/xwp/wordpress-develop/pull/232#issuecomment-304347911
Please review and amend a pull request I opened as opposed to submitting patch files. That should make it easier to collaborate and review. You can either fork the repo and open a PR into the trac-39686
branch, or I can add you as contributors to that repo to push directly.
This ticket was mentioned in Slack in #core-customize by westonruter. View the logs.
8 years ago
#76
@
8 years ago
@westonruter If you want to add me as a contributor it will be fine; otherwise, I will submit a PR for this soon. Thanks for pointing these issues. Just FYI, version 4.1.2 has been released and accessibility issues have been addressed, as well as other fixes
#77
@
8 years ago
@westonruter I have added my fixes to your PR. Let me know if I can assist you on anything else. Thanks
#78
@
8 years ago
- Keywords has-patch needs-testing added; needs-patch removed
Watchers: Please test. The patch can be tested via grunt patch:https://github.com/xwp/wordpress-develop/pull/232
#79
@
8 years ago
I'm still having trouble getting Vimeo to work (e.g. in the Video widget and the [video src="http://vimeo.com/..."]
shortcode), but strangely only in Twenty Fifteen and Twenty Sixteen: the video is just blank, but I can hear the audio. It plays properly in Twenty Seventeen. The oEmbed video player plays properly, however.
#82
@
8 years ago
I had a try too, works fine here.
https://github.com/xwp/wordpress-develop/pull/232#issuecomment-306130387
#83
@
8 years ago
@grosbouff @westonruter MEJS has been upgraded to 4.1.3 and it's now available in https://github.com/xwp/wordpress-develop/pull/232. Let me know if you need anything else
#84
@
8 years ago
This now needs wider testing.
Also, that last reference to Froogaloop does need to be removed, though we aren't sure the scenario where the code is invoked. It was introduced in [31016] for #29267. From reading the commit message, it seems it is supposed to be enqueued when adding a Vimeo to a post, but I can't reproduce that. @wonderboymusic do you recall?
#85
@
8 years ago
@westonruter Certainly. I will remove that conditional since I don't think it's needed at this point, unless it is proven to be needed. I have tried different ways to reproduce this without success. I'll commit that change tomorrow. Just FYI, 4.1.3 fixed mostly documentation and some things for HLS. Nothing that will really be that different from 4.1.2 in this context
#88
@
8 years ago
@grosbouff I think it's indicated as 4.8.1; has anybody of the watchers has tested the branch and come up with issues? @westonruter Can this go to 4.8?
#89
@
7 years ago
No. 4.8 is already been released. It needs wider testing and review by the media component maintainers before it can be committed. It's also not completely certain that it could be part of a minor release like 4.8.1 since minor releases may have restrictions on adding new files.
#90
@
7 years ago
The mediaelement-2.22.0 fixes @rafa8626 provided a month ago are not working any more in wp 4.8 with https and Safari on Mac OS 10.12.5 or mobile Safari on iOS 10.
Is there a chance to get at least a quick fix for the https issue, which is not fixed for many months now? I am still wondering why there are not more bug reports about it, are there still so few sites running on https?
#91
follow-up:
↓ 93
@
7 years ago
@westonruter Then why was the milestone set to 4.8.1 in a thread that is already 5 months old? Reported issues exist in the current media elements script that ships with wp 4.8. But WP does release new media widgets in which those reported issues are not fixed..... very frustrating i must say !
#92
@
7 years ago
If we can make the core contributors and whoever maintains the Media component to review and test this, and if you watchers give thumbs up to merging the branch described above, we will be good. I haven't seen anybody but @westonruter and @grosbouff testing this and I have addressed all that I can right now. But there must be more proof that it is good to go, so I'd ask you your help to close the loop on this and at the same time fix the bugs all of you have stated. I can guarantee you that I'm proactive and this branch is stable. Just help with testing and indicate here and in the Slack channel that it is being tested and approved.
#93
in reply to:
↑ 91
@
7 years ago
Replying to BackuPs:
@westonruter Then why was the milestone set to 4.8.1 in a thread that is already 5 months old? Reported issues exist in the current media elements script that ships with wp 4.8. But WP does release new media widgets in which those reported issues are not fixed..... very frustrating i must say !
Sorry you are frustrated. Issues get punted to future releases all the time when they miss the deadline for merge, when their patches are not completed and fully tested.
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
7 years ago
#95
@
7 years ago
@westonruter Upgraded to 4.2.2 version on current repository. Any updates on this? How can help to expedite the testing of this so it can be merged?
#96
@
7 years ago
- Milestone changed from 4.8.1 to 4.9
@rafa8626 I suggest raising in #core-media, getting component media maintainers involved in testing and reviewing the patch. Given the magnitude of the changes, and the presence of many new files (which complicate automatic upgrades), I expect that this will be something for the 4.9 major release and not the 4.8.1 minor release.
#97
@
7 years ago
@westonruter Thanks. Who should I ping on the #core-media channel, since I've tried to indicate that this should be testing but I haven't had any success? I understand that this one is a major change but is a necessary one
#98
@
7 years ago
Yes, I absolutely agree it is a major yet necessary change. The four component maintainers are listed at https://make.wordpress.org/core/components/media/
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
7 years ago
#101
@
7 years ago
- Keywords commit added
- Owner set to westonruter
- Status changed from new to accepted
OK, I'll move to commit this.
#103
@
7 years ago
- Owner westonruter deleted
- Status changed from accepted to assigned
OK, it's committed to trunk
. I'm leaving this ticket open, however, because it needs more testing and there are surely going to be some follow-up commits.
#104
@
7 years ago
@westonruter I released a new version that fixed issues for YouTube and improved the player, after some tests. Here's the PR: https://github.com/xwp/wordpress-develop/pull/246.
@grosbouff, @azaozz, @Plastikschnitzer, please review and test. Thanks
This ticket was mentioned in Slack in #core-media by desrosj. View the logs.
7 years ago
#108
@
7 years ago
- Keywords needs-testing removed
- Resolution set to fixed
- Status changed from assigned to closed
With [41320] I'll close this ticket. Any additional issues can be opened in new tickets.
#109
@
7 years ago
- Keywords needs-patch added; has-patch commit removed
- Priority changed from normal to high
- Resolution fixed deleted
- Status changed from closed to reopened
This change is causing JavaScript errors on the post editing screen.
Uncaught TypeError: Language code must have format `xx` or `xx-xx` at Object.s.language (mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12) at Object.o.5.15 (mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12) at i (mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12) at e (mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12) at mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12 s.language @ mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12 o.5.15 @ mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12 i @ mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12 e @ mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12 (anonymous) @ mediaelement-and-player.min.js?ver=4.2.5-74e01a40:12 wp-mediaelement.js?ver=4.9-alpha-40870-src:51
Uncaught TypeError: $(...).not(...).filter(...).mediaelementplayer is not a function at HTMLDocument.initialize (wp-mediaelement.js?ver=4.9-alpha-40870-src:51) at i (jquery.js?ver=1.12.4:2) at Object.fireWith [as resolveWith] (jquery.js?ver=1.12.4:2) at Function.ready (jquery.js?ver=1.12.4:2) at HTMLDocument.K (jquery.js?ver=1.12.4:2) initialize @ wp-mediaelement.js?ver=4.9-alpha-40870-src:51 i @ jquery.js?ver=1.12.4:2 fireWith @ jquery.js?ver=1.12.4:2 ready @ jquery.js?ver=1.12.4:2 K @ jquery.js?ver=1.12.4:2 wp-auth-check.js?ver=4.9-alpha-40870-src:101
Uncaught TypeError: Cannot read property 'hasClass' of undefined at HTMLDocument.<anonymous> (wp-auth-check.js?ver=4.9-alpha-40870-src:101) at HTMLDocument.dispatch (jquery.js?ver=1.12.4:3) at HTMLDocument.r.handle (jquery.js?ver=1.12.4:3) at Object.trigger (jquery.js?ver=1.12.4:3) at Object.jQuery.event.trigger (jquery-migrate.js?ver=1.4.1:633) at HTMLDocument.<anonymous> (jquery.js?ver=1.12.4:3) at Function.each (jquery.js?ver=1.12.4:2) at jQuery.fn.init.each (jquery.js?ver=1.12.4:2) at jQuery.fn.init.trigger (jquery.js?ver=1.12.4:3) at Object.<anonymous> (heartbeat.js?ver=4.9-alpha-40870-src:400)
Seen in Chrome 61 on macOS. Git bisected back to this commit.
#110
@
7 years ago
Correction: The above JavaScript errors are seen when using the admin area in a language with a three-letter locale code, such as Moroccan Arabic.
#111
@
7 years ago
Thanks @johnbillion . I'll prepare a PR to solve this. But can you create a separate ticket for this issue please? The overall task for this ticket was achieved already and now individual tickets should be created as @westonruter stated above
#112
@
7 years ago
- Keywords needs-patch removed
- Resolution set to fixed
- Status changed from reopened to closed
This ticket was mentioned in Slack in #core-media by desrosj. View the logs.
7 years ago
#115
@
7 years ago
@ocean90 I could but I don't know where this needs to be written, given that I've only had worked with GitHub in order to get the upgrade work wrapped up.
This ticket was mentioned in Slack in #core by flixos90. View the logs.
7 years ago
#117
@
7 years ago
@rafa8626 you could write the dev note here as a comment on Trac and we could copy it to the Make/Core blog for you.
#118
@
7 years ago
@rafa8626 if you want to draft the dev note in whatever tool/format works best for you, then I can help get that reviewed and published to Make/Core. Let me know if that works for you, thanks!
#119
@
7 years ago
@westonruter Thanks for the suggestion. @jbpaul17 here's my Dev Note:
MediaElement has been upgraded to 4.2.6, which includes, among lots of bug fixes: * Removal of jQuery in its core to make it more portable * UI with flexbox * Improvements in controls Accessibility * Stopped support for old browsers (supports IE11+, Chrome, FF, Safari, Android 4+ and iOS 8+) * Upgraded YouTube and Vimeo renderers to use new APIs (for Vimeo, it uses now https://player.vimeo.com/api/player.js, since Froogaloop is now deprecated) With this upgrade, a couple of things must be considered: 1. In order to create themes compatible with it, it is required to add the classPrefix: 'mejs-' in the player configuration, since latest version of MEJS switched to BEM naming convention, so make sure it is set correctly. 2. `player.options.mode` has been removed completely
#124
@
7 years ago
@rafa8626 @melchoyce @westonruter drafted dev note per @rafa8626's text; please review/edit: https://make.wordpress.org/core/?p=23897&preview=1&_ppp=1c0e03e83a
#125
@
7 years ago
I would add:
Support for iOS 10 and Mac OS on https-Websites added
We tested the latest nightly build of 4.9 and its working :-)
#126
@
7 years ago
I'll add as a 3rd consideration (just to give the users reassurance that nothing is gonna break):
3. Although jQuery was removed from the core package, a Mediaelement migration file has been included to provide full backward compatibility between themes and development of new features
The rest looks good to me
#127
follow-up:
↓ 128
@
7 years ago
Thanks @Plastikschnitzer and @rafa8626, I've added those items --> https://make.wordpress.org/core/?p=23897&preview=1&_ppp=07de112f5f.
@melchoyce @westonruter let me know if there's someone you'd like to review this before it's published... thanks!
#128
in reply to:
↑ 127
@
7 years ago
Replying to jbpaul17:
Thanks @Plastikschnitzer and @rafa8626, I've added those items --> https://make.wordpress.org/core/?p=23897&preview=1&_ppp=07de112f5f.
@melchoyce @westonruter let me know if there's someone you'd like to review this before it's published... thanks!
@bradyvercher Anything else you'd like to add? 👆
#38340 was marked as a duplicate.