[wp-trac] [WordPress Trac] #23282: Add shortcodes for inline HTML5 audio / video

WordPress Trac noreply at wordpress.org
Wed Mar 20 20:24:40 UTC 2013


#23282: Add shortcodes for inline HTML5 audio / video
-----------------------------+--------------------
 Reporter:  wonderboymusic   |       Owner:
     Type:  feature request  |      Status:  new
 Priority:  normal           |   Milestone:  3.6
Component:  Media            |     Version:  trunk
 Severity:  normal           |  Resolution:
 Keywords:                   |
-----------------------------+--------------------

Comment (by wonderboymusic):

 Replying to [comment:63 obenland]:
 > * my .m4a file was passed to the shortcode as `mp3="path/to/file.m4a"`,
 thus failing the filetype check in the audio shortcode.
 [[BR]]
 Did this happen using Insert Into Post? that shouldn't happen:
 https://core.trac.wordpress.org/browser/trunk/wp-includes/js/media-
 editor.js#L99
 I have opened up a ticket for Insert Into Post issues: #23831
 [[BR]]
 [[BR]]
 > * I'd always get the following JS warning: "Specified "type" attribute
 of "audio/mpeg" is not supported. Load of media resource
 http://wpthemetestdata.files.wordpress.com/2008/06
 /originaldixielandjazzbandwithalbernard-stlouisblues.mp3 failed." and "All
 candidate resources failed to load. Media load paused."
 [[BR]]
 That MP3 worked for me...
 [[BR]]
 [[BR]]
 > * for some reason the container div (.wp-audio-shortcode) has it's
 height set to 34px (width: 454px) in 2013's audio post format (it should
 be 30px). It has the correct height and in standard post formats (width:
 400px). Where can I find the code that deals with that?
 [[BR]]
 Not sure why it is getting set to 34px, but:
 https://core.trac.wordpress.org/browser/trunk/wp-
 includes/js/mediaelement/mediaelementplayer.css#L112
 https://core.trac.wordpress.org/browser/trunk/wp-includes/js/mediaelement
 /mediaelement-and-player.js#L1841
 [[BR]]
 > It's a nit pick, but based on the singular in the function name, I
 didn't expect more than one result from `get_post_audio|video()`.
 yeah... `get_post_videos()` would make sense there but not
 `get_post_audios()`

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/23282#comment:65>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list