I thought I saw some implementation of this going on in dev but couldn't find an issue...marking major since it enables the user to select view modes that aren't setup or intended to be used.

When inserting media via the wysiwyg file browser, the view modes the user is able to select should be limited to only the modes enabled for the file type. For example if an 'application' file is chosen in the media browser, the next selection of view mode should be limited to the enabled view modes at /admin/structure/file-types/manage/application/display (under custom display settings).

CommentFileSizeAuthor
#8 1632502-wysiwyg-limit-view-modes.patch2.89 KBdave reid
#7 1632502-wysiwyg-limit-view-modes.patch2.82 KBdave reid
#5 1632502-wysiwyg-limit-view-modes.patch2.68 KBdave reid

Comments

Rob_Feature’s picture

Rob_Feature commented
Title: Limit View Modes to Enabled » Limit WYSIWYG View Modes to Enabled Modes

Clarifying title....

drzraf’s picture

drzraf commented

probably related #1421444: Does media_format_form() selects file styles the right way ?

devin carlson’s picture

devin carlson commented

Marked #1637310: File Types-> Manage Display ineffective after configuration. as a duplicate.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Agreed. Taking a look at this also makes me think there should be a 'default' view mode available as well.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 1632502-wysiwyg-limit-view-modes.patch2.68 KB

Patch attached for review.

drzraf’s picture

drzraf commented

It partially works, almost every view_modes are still available.

In my case I disabled file_styles_square_thumbnail but custom settings from entity_get_info('file') is still marked as TRUE.
Moreover, by only calling entity_get_info('file') how would you differentiate between an image for which a view_mode has been customized an another file bundle for which it hasn't been ?

Anyway, the only view_mode I never customized (token) disappeared after I applied the patch.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 1632502-wysiwyg-limit-view-modes.patch2.82 KB

I wasn't checking the proper function. Revised patch for review.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 1632502-wysiwyg-limit-view-modes.patch2.89 KB

Needed to be statically cached by file type.

drzraf’s picture

drzraf commented

working, thanks !

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Committed #8 to Git:
http://drupalcode.org/project/media.git/commit/2b2acd6 (7.x-2.x)
http://drupalcode.org/project/media.git/commit/76e7f47 (7.x-1.x)

Status: Fixed » Closed (fixed)

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