Closed (duplicate)
Project:
Popups API (Ajax Dialogs) [D7]
Version:
6.x-2.0-alpha5
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Anonymous (not verified)
Created:
30 Mar 2009 at 01:53 UTC
Updated:
16 May 2024 at 00:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
mstef commentedWeird..
It works when using the add + reference but not the regular popup links...
Also the collapsible regions follow the same trend..
What's the difference? How do you get em all to work?
Comment #2
lelizondo commentedI'm also having the same problem but only with IE6 & IE7.
Comment #3
mstef commentedSame in firefox (v.3, ubuntu)
Comment #4
lelizondo commentedthere's also a usability issue, can't say it's a bug.. I usually start typing a few letters until the autocomplete starts and gives me choices.. because my both hands are on the keyboard I usually move with the keyboard until I find the right option and then hit enter.. well, if I do this with popups the form it's submitted because of the enter... that's not right..
I don't know if anyone has this problem..
Comment #5
mstef commentedThe popup add & reference module has autocomplete and collapsible regions working perfectly. From what I've read, I believe it's because it includes the needed javascript files which seems like it can only be done from within the module. I'm going to look into it more and perhaps it can be replicated from nodes via php.
Comment #6
mstef commentedalright alright i know i'm the best...
just add this before the link or however you want to implement it..
Comment #7
frankcarey commentedDo you mean that this JS has to be loaded on the page the popup link is on, or the html inside the popup?
Comment #8
mstef commentedRead the documentation on the module. Create a dummy module to hook the popups.
Comment #9
frankcarey commentedcool, thanks for the workaround!
Do we know why popups is't loading the js on it's own?
Comment #10
mstef commentedit's not a work around..that's how you tell popups what to load...you can load additional css too...
Comment #11
frankcarey commentedso privatemsg (in this case) should be loading it's own js / css with this hook to be correctly integrated, but we can make a custom module instead for modules that do not yet support popups?
Comment #12
mstef commentedNot really but I understand your point...
Comment #13
greg.harveySlightly OT, but relevant nonetheless. The approach in #8 isn't documented in the current popups README.txt and I can't find any further reference to documentation on the project page. Can anyone point me to some proper popups docs? =)
Comment #14
frankcarey commentedadditionalJavascript and additionalCSS properties only work for 1.3... it has since been removed from the 2.0 branch, but there is additional code now so that popups should do this on it's own. I am only using 1.3 right now, so i don't know how well it's working. not sure if these 2 properties are documented in 1.3 or not.
Comment #15
greg.harveyHmm, strange. I'm using 6.x-2.0-alpha5 and it isn't loading my JavaScript, hence me looking in to the hook. Sounds like I need to raise a bug report, but I'll try and work out what's happening first.
Comment #16
kenorb commentedThe same problem.
I'm using nodereference field.
Comment #17
partap commentedI'm not sure if this is exactly the same bug...depends on what is meant by "don't work".
In Firefox, the autocomplete fields in my popups technically do work, but they are covered up by the browser's own autofill pulldown.
e.g.
When I load my form on a normal page, the field has a white background and autocomplete works as expected. But, if I load the form in a popup, the field has a yellow background and typing in it brings up the browser's Form Autofill suggestions.
However, if I have a big enough autocomplete list, I can see it sticking out from behind the browser's autofill list, so I can see that it's actually working, but for some reason the browser won't disable it's own pulldown. I tried the drupal_add_js() suggestion but it makes no difference.
Sound familiar to anyone?
Comment #18
_tychris commentedIt seems that problem coming from file "misc/autocomplete.js" on the line 13, the variable "context" isn't add.
Before :
var input = $('#' + this.id.substr(0, this.id.length - 13))
After :
var input = $('#' + this.id.substr(0, this.id.length - 13), context)
You can see a topic about same problem : http://drupal.org/node/630424.
Comment #19
kenorb commented#630424: Consider 'context' in more core JS files