When the tab includes a view, the jquery expression line 56 (viewsdsiplaytabs.js)
var parentView = $(elem).parents().find('div.view').attr('class');
has a jquery find call, which returns all the div elements having a view class attribute, while the expected behavior is to get only the container element of the tab link element. In other words, this call is heavier, and make bugs when our tabs include other div.vew elements
so you should replace it by the following:
var parentView = $(elem).parents().filter('div.view').attr('class');
it's faster to execute obviously, and works everywhere I could test it.
Best regards

Comments

solipsist’s picture

Would you consider packaging this as a patch instead?

raphael waeselynck’s picture

Well I'm really busy these times, so it will not be before a long time. But do you think a patch is usefull for only one line ? Sorry if my question seems stupid, I'm not really used to this way to deal with issues...

jordanmagnuson’s picture

Have confirmed that this patch solves some problems with this (great) module... should be rolled.

solipsist’s picture

rwaesely: I'm insanely busy too - hence the request.
davidlerin: I'd roll if if it were an actual patch :)

solipsist’s picture

Status: Needs review » Fixed

Fixed.

solipsist’s picture

Status: Fixed » Closed (fixed)