Investigate using CTools rather than Lightbox for popups/AJAX

I believe that modalfram api (http://drupal.org/project/modalframe) is now in drupal 7 core, so perhaps that's what should be used for popups?

Have a look at Dialog API. It's all the goodness of CTools modals, but uses jQuery UI dialogs. Its actually based on the work I wrote up in the blog post mentioned in the first post.

FWIW, XML Sitemap just integrated Dialog (#661780: AJAX-ify the type settings on admin/settings/xmlsitemap) and there is an issue in Skinr to use Dialog (#690862: Use Dialog API module instead of Modalframe/Overlay).

modalframe is more like the Overlay in D7. If you want lighter popups, I'd probably recommend the Dialog API.

I think actually a lighter weight popup may not make as much sense for this module as the modalframe api. Modalframe adds an iframe to the page to render the window in which I think makes a bit more sense when you're rendering new forms in the popups (as contact importer does). Also, it's possible that at some point there would be multi-step forms in there, depending on the providers needs.

This patch add modalframe API support through automodal module.
Automodal allows adding modalframe to modules in the same way than popup.

The patch for now only use the modalframe if there is no lightbox2 installed.
Probably the best it would be to add an admin option to select one of them.

i would vote for modalframe OR ctools, but modalframe slightly more. Dialog is not good atm in my eyes. to many problems and not enough work beeing put in it.

Will do, only thing is that i don't use this module itself. Just found it, and alltough it is a drupal diamont i don't need it in my recent project. BUT i will test it.

against what is this patch made?

$ patch < patch.patch
patching file contact_importer.module
Hunk #1 FAILED at 270.
Hunk #2 succeeded at 575 with fuzz 2 (offset 297 lines).
Hunk #3 succeeded at 827 with fuzz 2 (offset 337 lines).
Hunk #4 FAILED at 879.
2 out of 4 hunks FAILED -- saving rejects to file contact_importer.module.rej

when i do it against the 6.x-2.x-dev of 2010-Sep-15.

aha ok i will test the dev.

Octazen has been purchased by facebook, http://octazen.com/ http://techcrunch.com/2010/02/19/octazen-what-the-heck-did-facebook-just...

Ok, but i guess the octazen refering link on the project page will have to be removed, since it is not possible anymore to download it. And i don't see facebook buying a opensource company for the fact that it's open and the contributors probably would leave. If facebook would buy drupal for 100mil and you wouldnt get a piece of the pie, would you still be as active as you are now?

But back to this, i will test this tomorrow, time for sleep now.

i dont know if this is a strange question, but why would one install contact_importer when OpenInviter has also a drupal module?

Im sorry i am not able to help you with testing this, i am not going to use this module, atleast for the next 6months, but it's giving me quite the hassle.

1) Unable to run postinstall.php evne with 265MB
2) I guess this is because of 1, i dont see a list of providers. So im unable to select one.

1) i have no SSH atm.

Finally I could test the new dev release and it works fine with the modalframe API through automodal.
Thank you.

First install ctool module (Drupal 7)
Using ctool show user detail in popup...Here is the code...

function mymodule_menu() {
	
$items['preview-profile/%ctools_js/%user'] = array(
 'title' => 'Preview Profile Page',
		'page callback' => 'user_preview_profile',
		'page arguments' => array(1,2),
		'access callback' => TRUE,
);

return $items;
}
function user_preview_profile($js = NULL, $user) { 
 if (!$js) {
		return drupal_access_denied();
	}

	$title = 'Profile Preview';
	$output = drupal_render(user_view_page($user, 'full'));
	if ($js) {
		ctools_include('ajax');
                ctools_include('modal');
		$commands = array();
		$commands[] = ctools_modal_command_display($title, $output);
		print ajax_render($commands);
		exit;
	}
	else {
		return $output;
	}
}
//And to create link....

   ctools_include('modal');
   ctools_include('ajax');
   ctools_modal_add_js();
ctools_modal_text_button(t('Preview Profile Page'), 'preview-profile/nojs/'.arg(1), t('Preview Profile Page'), 'ctools-modal-ctools-ajax-register-style');