[#16237] - [4.0] Update mediaelement.js to v4.1.1
- Fixed in Code Base
- 28 May 2017
- Medium
- Build: 4.0-dev
- # 16237
- Diff
- C-Lodder:mediaelement
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
We're currently 2 major versions behind on mediaelement.js, so this PR updates it to the latest version.
As it stands, there is still some jQuery behind the scenes but I won't remove that as I'm sure the new media manager will sort it.
Testing Instructions
- Go to the Media Manager options
- Add
video/mp4as a Legal MIME Type - Add
mp4as a Legal Extension - Upload an MP4 video (can use this sample one)
- Click the video to play
Expected result
Video plays as expected.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_media Repository JavaScript |
Trolls please take a note of the common patterns...
#dgt41 You seriously should start watching your language here. Not everyone disagreeing with you is automatically a troll.
Guess this pr should be done against the media manager repo, it affects some views we don't have anymore in the new media manager.
How and where is this lib used?
Do we use that lib in the new MM as well?
We haven't implemented a preview for videos yet, so atm no
| Labels |
Added:
?
|
||
| Category | Administration com_media Repository JavaScript | ⇒ | Repository JavaScript |
For me it is ok to merge. We didn't decide right now how preview is done.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-28 14:00:24 |
| Closed_By | ⇒ | wilsonge |
Must be a world wide disease that everyone want to move away from jQuery🎉
Trolls please take a note of the common patterns...