I tried to hide the format selector in the media_format_form with the hook_media_format_form_prepare_alter hook, but then the JavaScript failed because it assumes the format selector is a select.

Changed media.format_form.js to allow any type of input.

CommentFileSizeAuthor
#2 simplify-format-selector-js-1960936-2.patch663 bytesdevin carlson
format-selector-x-0.patch858 byteslogaritmisk

Comments

slashrsm’s picture

slashrsm commented
Status: Needs review » Needs work

Looks OK. It would probably be better to select "#edit-format", as it should be faster.

See (Additional notes): http://api.jquery.com/input-selector/

devin carlson’s picture

devin carlson commented
Component: WYSIWYG integration » Code
Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new simplify-format-selector-js-1960936-2.patch663 bytes

As slashrsm mentioned in #1, it would be faster to select #edit-format.

devin carlson’s picture

devin carlson commented
Status: Needs review » Fixed

Tested #1 with an existing Media install and verified that the format form's format selector continued to function properly.

Committed #1 to Media 7.x-2.x.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.