Currently, the modalBackdrop color and opacity are hardcoded into modal.js, making it impossible to theme. I added those options to the Drupal.settings.CToolsModal object -- in line with how the closeText, closeImage, and throbber work. Here's the change:

    var opts = {
      // @todo this should be elsewhere.
      opacity: Drupal.settings.CToolsModal.backDropOpacity,
      background: Drupal.settings.CToolsModal.backDropColor
    };

...and then reflected in modal.inc

  $settings = array('CToolsModal' => array(
    'closeText' => t('Close Window'),
    'closeImage' => theme('image', ctools_image_path('icon-close-window.png'), t('Close window'), t('Close window')),
    'throbber' => theme('image', ctools_image_path('throbber.gif'), t('Loading...'), t('Loading')),
    'backDropOpacity' => '.55',
    'backDropColor' => '#fff'
  ));
CommentFileSizeAuthor
modal-opts.patch1.23 KBjasonn1234
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sdboyer’s picture

Status: Needs review » Fixed

Makes total sense to me. Committed. Thanks!

Status: Fixed » Closed (fixed)

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

geerlingguy’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Closed (fixed) » Active

Was this ever forward-ported to 7.x-1.x?

merlinofchaos’s picture

It should have been, we did a merge late in 2010. Can you verify it's not in 7.x?

geerlingguy’s picture

Status: Active » Fixed

Nevermind, it was my own dumb code not working... :)

Status: Fixed » Closed (fixed)

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