Popups to add panes/context/anything else not working (Patch included)
sarvab - January 24, 2008 - 01:13
| Project: | Panels |
| Version: | 5.x-2.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
I'm not sure if my version of jQuery is newer than from jquery_update (must be), but it seems that the .top and .left jQuery functions have been removed. This problem would prevent loading the popup in any circumstance, sometimes giving an error JS alert or sometimes not doing anything.
This patch just replaces those two functions. Even if it works for most people it would probably be a good idea for future jQuery updates and if anyone is currently having a problem with popups it may be worth trying.
| Attachment | Size |
|---|---|
| mc.js_.patch | 2.27 KB |
#1
Still there in -dev, sounds critical to me.
#2
Tried this patch, ran update.php but still getting same error message:
http://dev.domain.org/panels/ajax/add-content/1/row_1_1
missing ; before statement
jquery.js (line 2)
This is a clean intial install of Panels 2 beta2.
#3
This patch only matters for people who have a newer version of jquery than ships with Drupal.
#4
Wouldn't call it critical then, but we should take another look at it before shipping the final.
#5
This patch appears to ahve already been applied. Perhaps it was applied as another patch.
#6
Automatically closed -- issue fixed for two weeks with no activity.