Without this, DHTML menus, CSS menus, etc will be underneath the applet.

function mollom_reports_page() {
  $embed_attributes = array(
    'src' => 'http://mollom.com/statistics.swf?key=' . check_plain(variable_get('mollom_public_key', '')),
    'quality' => 'high',
    'width' => '100%',
    'height' => '430',
    'name' => 'Mollom',
    'align' => 'middle',
    'play' => 'true',
    'loop' => 'false',
    'allowScriptAccess' => 'sameDomain',
    'type' => 'application/x-shockwave-flash',
    'pluginspage' => 'http://www.adobe.com/go/getflashplayer',
    'wmode' => 'opaque', // Add this so DHTML menus and such will go OVER the Flash element
  );
// ...

Not always with embedding this way because is not valid XHTML (or HTML for that matter) but nobody really validates the back-end HTML so I guess it's okay here.

More information about wmode (window mode):
http://www.communitymx.com/content/article.cfm?cid=E5141

CommentFileSizeAuthor
#1 mollom-HEAD.stats-wmode.1.patch633 bytessun

Comments

sun’s picture

Version: 6.x-1.9 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new633 bytes

Actually, it needs to use wmode 'transparent' to make browsers apply a z-index on it. See #195386: Menu under flash content for detailed debugging information.

sun’s picture

Title: Add wmode => opaque for SWF on the reports page » Add wmode attribute to Mollom statistics SWF
dries’s picture

Status: Needs review » Reviewed & tested by the community

This looks good to me. Feel free to commit, sun.

sun’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for reporting, reviewing, and testing! Committed to all branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

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

  • Commit fce8eae on master, fai6, 8.x-2.x, fbajs, actions by sun:
    #922336 by sun, Andy Udvare: Fixed missing wmode attribute on Mollom...

  • Commit fce8eae on master, fai6, 8.x-2.x, fbajs, actions by sun:
    #922336 by sun, Andy Udvare: Fixed missing wmode attribute on Mollom...