navigationFilter is broken

[obecker]

The navigationFilter of ui.accordion is broken since it always selects <a> elements by default (which matches the default for the header option).

Line 50 currently is

var current = $(container).find("a").filter(options.navigationFilter);

it should be

var current = $(container).find(options.header).filter(options.navigationFilter);

[Jörn Zaefferer]

The navigation-option may need some attention, but your proposed fix is invalid, too. For a click-accordion, links inside headers can't be visited.

The navigation option looks for all anchors to find the one that matches the current page, then activate the parent panel of that anchor.

Looking for just headers wouldn't provide any benefit.

[obecker]

It may well be that I don't understand the entire logic behind the accordion (being a JQuery newby, though), however we changed the default <a> header to a <span> and with the provided fix this works like a charm:

$('#foobar').accordion({
   active: true,
   alwaysOpen: false,  
   header: "span.title",
   navigation : true,
   navigationFilter: function() {
       return $(this).hasClass('fooId#{param['fooId']}');
   }
});

(#{param['fooId']} evaluates to the fooId URL parameter)

[Jörn Zaefferer]

#4653 is related. Also, I see no reason to change the API of navigationFilter - for anything that can't be matched via navigationFilter, use the active-option instead.