Great module - will come in really useful. Looking forward to seeing it develop.

Discovered a couple of minor bugs relating to the ajax_loader graphic.

I think

drupal_add_css(drupal_get_path('module', 'simple_dialog') . '/css/simple_dialog.js');

should be:

drupal_add_css(drupal_get_path('module', 'simple_dialog') . '/css/simple_dialog.css');

--

Also in the module's css file

background: url(ajax_loader_white_bg.gif) no-repeat center center;

should be:

background: url(./images/ajax_loader_white_bg.gif) no-repeat center center;

Cheers

Luke

Comments

drclaw’s picture

Thanks for the info. I'll fix this in the next commit.

drclaw’s picture

Status: Needs review » Active

This has been fixed and committed. Thanks again!

drclaw’s picture

Status: Active » Fixed

oops. Fixed

Status: Fixed » Closed (fixed)

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