When using http://drupal.org/project/popups if viewscarrousel is used on the maincontent (inot in the popup) it generate an exception.
Modifiing viewscarousel.js correct the problem:
(function($) {
Drupal.behaviors.viewscarousel = function(context) {
if (!Drupal.settings.viewscarousel) return;
$.each(Drupal.settings.viewscarousel, function(id) {
if (this.scroll) this.scroll = parseInt(this.scroll);
if (this.start) this.start = parseInt(this.start);
if (this.visible) this.visible = parseInt(this.visible);
if (this.auto) this.auto = parseInt(this.auto);
$('#' + id).jcarousel(this);
});
}
})(jQuery)
Comment | File | Size | Author |
---|---|---|---|
#5 | viewscarouselpatch.patch | 423 bytes | parazit_vip |
Comments
Comment #1
RobLoachThis is in 6.x-1.x-dev.
Comment #2
asak CreditAttribution: asak commentedAs far as i can tell, there is still a conflict with Popups API.
I tried all possible combination of Views carousel versions and popups api version, with and without jquery update, but the end result in all cases is:
On pages in which views carousel is loaded - the popups do not function correctly.
We got popups to work flawlessly on all other pages, but on the pages we display a carousel we're experiencing the following:
1. When commenting in a popuped node, while on a page without a carousel, the pop reloads with the new comment. When trying the same from a page with a carousel - The whole page redirects to the new node.
2. Same goes for validation of popuped node creation forms.
I found that i get all kinds of error in jquery.js in different combinations of the modules i mentioned - but the end result is always the same.
Comment #3
BLadwin CreditAttribution: BLadwin commentedI'm having a similar issue, but it's when I am displaying a view carousel inside the popup modal. The carousel displays fine, but when the "next" button of the pager for the carousel is activated, the popup doesn't refresh, it brings me to the actual page where the carousel is displayed. Just wanting to know if there is a fix available for this issue.
Comment #4
mfer CreditAttribution: mfer commentedI just installed popups and was testing this. I could not replicate the problem. Could someone provide the modules to enable and the configuration to repeat finding the error?
Comment #5
parazit_vip CreditAttribution: parazit_vip commentedThis patch fixed this problem.
Comment #6
nathanjo CreditAttribution: nathanjo commentedI experience this when js aggregation is enabled. The arrows are not being loaded in popups module.
Any fix for this?