-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Embed block doesn't use MediaElement.js #4961
Comments
maybe this is also related #4842 |
This just uses the HTML5 video tag. Any reason using mediaElement is better than using a standard? |
I use it to add speed buttons for .5x, 1x, 1.5x, 2x speeds, which rely on a html5 As for WP itself using mediaElement.js IDK. I do know that they recently bumped versions rather than choosing to remove it. |
Oh https://wordpress.org/support/topic/a-godsend-6/ got a review on my plugin (first public plugin review) If this isn't supported I will have to make a separate plugin (because it's not right to enforce in the existing) to bring mediaElement.js to Gutenberg as a block for video's so that this user can continue to be happy with my plugin 😉. If I have to do that, I'll probably deactivate the existing blocks so there is not confusion around "which is the right block", which seems an unnecessary thing to have to document. |
Another reason to use it is that it aims to present a consistent UI across browsers. Separating from them would be an indication that is not a design goal, or that duplicated efforts have been chosen. There is feedback from a user of my media-playback-speed plugin (not perfect). They are using media-element.js with a niche plugin to allow playback speed to alter, enabling students to adjust from beginner, to proficient. (not my design work, and I know it's audio not video btw) |
There is work being done in #805 to bring mediaelement to Gutenberg. I believe this should be done as a followup PR to these changes. |
I've created an issue in #5240 to track my PR for the |
@anthonyburchell please see my comments on that issue. There may be other reasons to add a mediaelement component, but #4710 does seem like some of the work being done could benefit both this and other efforts (another plugin I have 😏) without too much JS or frontend engineering |
Any update here? I wonder how there's no update on this vital feature. By vital I mean video thumbnail generation from both local and video websites that could be easily passed to the social share buttons... What is the status? The best solution is either of a usage of a third party videoplayers which makes no sense anymore or (used to be) a WP plugin (video thumbnails) not updated in the last 3 years. And having in mind that Gutenberg is not from yesterday.... Let us even not mention WP... Sooooo. Please, please please. what is the status of this bug...feature? |
@wetfirer there is a plugin you can use to preserve shortcode functionality https://wordpress.org/plugins/cd2-gutenberg-shortcode-preview-block/ So long as WP keeps their shortcodes supported (interim solution), and keeps Gutenberg version stable this will work with other mediaElement / traditional media plugins, providing the visual nature of Gutenberg, with familiar controls I use it along with https://wordpress.org/plugins/media-playback-speed/, a plugin which I re-wrote and published to the plugin directory. |
Checking in. @ntsekouras Nik I think you might want to know about this issue. |
Browser players have come a long way since |
Issue Overview
Video widget seems to differ from traditional insert of video from media library. Does not use video shortcode and does not use mediaElement.js as far as I can tell.
Steps to Reproduce (for bugs)
Expected Behavior
Current Behavior
Possible Solution
I don't know. Parity with existing video media seems reasonable, but maybe there are things I don't know.
Screenshots / Video
Related Issues and/or PRs
N/A
Todos
The text was updated successfully, but these errors were encountered: