When admin_menu is enabled, it gets displayed on the Media popup page render. I filed #914786: Best way to disable admin_menu on iframe popups to figure out a long-term solution for this type of problem, but for now adding admin_menu_suppress() fixes the issue.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 914834-media-adminmenu-suppress-D7.patch | 1.92 KB | dave reid |
Comments
Comment #1
dave reidComment #2
dave reidScreenshot of popup after patch: http://skitch.com/davereid20/dagc1/create-media-localhost
Comment #4
robeano commentedThis seems reasonable for a temporary fix until #914786 is handled. Committed.
Comment #5
tsvenson commented@robeano
This fix causes fatal error when using the WYSIWYG media browser and Admin Menu is not enabled. There is no check to test if Admin Menu is available or not.
Comment #6
dave reidGah, can't believe I didn't add that check.
Comment #7
robeano commentedHow embarrassing. I committed a check that Admin Menu is enabled before suppressing the extra menu.
Comment #8
robeano commentedChanging issue status to Fixed.
Comment #9
gandhi commented#1: 914834-media-adminmenu-suppress-D7.patch queued for re-testing.
Comment #11
dave reidComment #13
RobW commentedIssue title: woah.