We've had several reports about people changing the 'Preview' view mode display and file display settings and it messing up the Media browser because it output unexpected code. I think we should use media_help() to display some small warning about the view mode affecting other systems.

CommentFileSizeAuthor
#1 1664504-preview-view-mode-change-warning.patch1.1 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
1664504-preview-view-mode-change-warning.patch1.1 KB

Patch attached for review along with a screenshot.

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Reviewed & tested by the community

The patch in #1 applied cleanly and added the warning message depicted in the screenshot to both the manage display and manage file display preview pages.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Reviewed & tested by the community » Fixed

Committed #1 to Git.

http://drupalcode.org/project/media.git/commit/8859d75 (7.x-2.x)
http://drupalcode.org/project/media.git/commit/1f7e661 (7.x-1.x)

Status: Fixed » Closed (fixed)

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