During the process of embedding a file into a textarea using Media's WYSIWYG integration, Media: BlipTV currently checks to see if the file being embedded has the mimetype video and then "bypasses" the view mode selection form by automatically submitting the form.

This functionality should be removed for a number of reasons:

  • It limits the user from displaying multiple versions of their content (a "large" video on one page and a "preview" on another).
  • It's confusing to the user; why is a view mode selector available for every other type of media.
  • The ability to automatically submit the view mode form may be removed by #1301774: Remove some legacy popup browser code.

This is also currently not working since Media: BlipTV checks for the mimetype video which is incorrect (it should be something like video/bliptv) and may not be available (see #1823078: Create video/vimeo mimetype).

CommentFileSizeAuthor
#1 remove-view-mode-autosubmit-1824156-1.patch612 bytesdevin carlson

Comments

devin carlson’s picture

devin carlson commented
Status: Active » Needs review
StatusFileSize
new remove-view-mode-autosubmit-1824156-1.patch612 bytes

A patch to remove the view mode autosubmit.

Rob_Feature’s picture

Rob_Feature commented

I totally concur...we need to be able to choose the view mode for videos. Autoselecting one is fairly user-hostile.