Closed (fixed)
Project:
Simple Dialog
Version:
7.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Mar 2012 at 14:54 UTC
Updated:
7 Jul 2012 at 04:34 UTC
I tried both the 7.x-1.2 and latest dev versions and get the same on both: When I add a link, the dialog that pops up has empty content. I added a custom class in the settings called 'popup' which successfully launches the popup, but it's empty.
Here's an example of my link:
<a id="Google" title="Google" name="Google" href="http://google.com" class="popup">test</a>
Any hints? Marking critical because, well...nothing works for me :)
Comments
Comment #1
drclaw commentedSorry, I should have documented that you can't use external links in the dialog. The link needs to be to a page on your site. This probably won't change either since the dialog uses ajax to load the target page.
What you need is an iframe pop-up (which I was specifically trying to avoid). I think there is some neat stuff you can do with lightbox (http://drupal.org/project/lightbox2) though... maybe take a look at that?
Hope this helps!
Comment #2
Rob_Feature commentedI just tried popping up an onsite file and it did the same thing...anything wrong with this?
Comment #3
drclaw commentedPossibly. So, the popup targets a particular element on the target page, by ID. This is handled through the "name" attribute.
<a title="View" name="content" id="View" href="/node/1" class="popup">Popup</a>The above example will pull the element with the id "content" from node 1, and put it in the dialog. The reason yours is coming up empty is that it's searching the node/1 page for an html element with the id "View" and coming up with nothing.
Comment #4
Rob_Feature commentedAhhh, I wasn't understanding the use of the 'name' element (is that in the documentation somewhere?). I'll try again with that specified and report back.
Comment #5
drclaw commentedIt looks like I've got that info on the project page and in the help file, but the README needs an update! I'll fix that ASAP. Thanks!
Comment #6
drclaw commentedThis should be clearer in the documentation now.