Conflict with popups [#514528]

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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

RobLoach

he/him

CreditAttribution: RobLoach commented 14 July 2009 at 23:37

Version:

6.x-2.x-dev

» 6.x-1.x-dev

This is in 6.x-1.x-dev.

Comment #2

asak CreditAttribution: asak commented 14 January 2010 at 21:10

As 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 commented 5 March 2010 at 21:36

I'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 commented 9 August 2010 at 21:38

Status:

Active

» Postponed (maintainer needs more info)

I 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?