Closed (fixed)
Project:
Views Display Tabs
Version:
6.x-1.0-beta4
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
21 Dec 2009 at 14:18 UTC
Updated:
25 Feb 2010 at 08:17 UTC
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
Comment #1
solipsist commentedWould you consider packaging this as a patch instead?
Comment #2
raphael waeselynck commentedWell 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...
Comment #3
jordanmagnuson commentedHave confirmed that this patch solves some problems with this (great) module... should be rolled.
Comment #4
solipsist commentedrwaesely: I'm insanely busy too - hence the request.
davidlerin: I'd roll if if it were an actual patch :)
Comment #5
solipsist commentedFixed.
Comment #6
solipsist commented