Make WordPress Core

Opened 4 years ago

Last modified 7 months ago

#41991 new defect (bug)

MediaElement CSS only brings in broken "legacy" css by default??

Reported by: programmin Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version: 4.9
Component: Media Keywords: close
Focuses: ui Cc:


The mediaelementplayer-legacy.min.css mentioned in #39686 should probably include the mediaelementplayer.css as that is what actually has the css for

.mejs__time-rail, .mejs__mute

and others that mejs adds to the page (why did they change the class names??)

Otherwise the mejs player is a broken bunch of odd looking buttons.

Change History (8)

#1 @rafa8626
4 years ago

@programmin You need to use the element classPrefix in the MediaElement.js configuration to avoid breaking things:

classPrefix: 'mejs-'

#2 @rafa8626
4 years ago

If you check wp-includes/script-loader.php file, line 453, you will see that element used for the Embed element

#3 @rafa8626
4 years ago

  • Keywords close added
  • Severity changed from normal to minor

#4 @programmin
4 years ago

So existing themes will need to change that in configuration js somewhere to have the current 4.8 behavior?

Last edited 4 years ago by programmin (previous) (diff)

#5 @rafa8626
4 years ago

My assumption is that, unless you are using MEJS without the Embed widget, yes, it will be required to add that flag. Otherwise, it is covered already

#6 @programmin
4 years ago

Why should I change the prefix when the default as seen on their site is prefixed mejs ? http://www.mediaelementjs.com/

I really think this is a bug especially since the mediaelementplayer.css included doesn't seem to be mentioned anywhere in the WP code.

#7 @rafa8626
4 years ago

Because that prefix is the one used by most of WP products; if we don't set it up we'll break a lot of code and themes. As an agreement the new code will use BEN notation but for backward compatibility the prefix must be used. It's not a bug since the intention was not to use the new style sheet for WP and at the same time provide the latest and more stable version of MEJS without disturbing a lot of things

This ticket was mentioned in Slack in #core-media by sabernhardt. View the logs.

7 months ago

Note: See TracTickets for help on using tickets.