The patch fixes the 2 problems above, i.e. title without the <a> tag and it does not expand after <a> tag is added.

Using panels 5.x-2.x-dev (2.x-beta2)

panels_accordion.module (Line 46):

    $pane->subject = '<a href="#" ' .$class. '>' .strip_tags($pane->title, '<p><h1><h2><h3><strong><img>'). '</a>';

to

    $pane->title = '<span><a href="#" ' .$class. '>' .strip_tags($pane->title, '<p><h1><h2><h3><strong><img>'). '</a></span>';

Comments

Veggieryan’s picture

Veggieryan commented

+1 for this.

works perfectly for me in panels 2 beta 2.

this closes my other issue: http://drupal.org/node/211036

davidburns’s picture

davidburns
he/him
Primary language English
Location Philadelphia
commented

I've updated this to work with the newest version of Panels (beta 3).
Tested w/ fresh install of Drupal, Panels, and Panels accordion

When panels gets a final release I'll do a final release of this module as well.

davidburns’s picture

davidburns
he/him
Primary language English
Location Philadelphia
commented

Newest version of Panels removed the span around titles... I've hardcoded it back into the module so jQuery could Identify the links as expandable

apratt’s picture

apratt commented

Using Panels beta 4b and definitely having this problem but all the changes seem to be incorporated into the 5.x-1.x-dev . I imagine something else has changed.

I started to work through the code and then realized that there might be a quick answer that somebody more familiar with jquery and Panels 2 development had.

I'll go back to work on it tomorrow and post if I find anything...

apratt’s picture

apratt commented

Updating to panels beta5 seemed to solve the problem.

Angus

leo.ruffini’s picture

leo.ruffini commented

Hi,

Using Panels2.0-rc1a and Panels Accordion 5.x-1.0-beta and also having this problem.

I installed the module and followed the instructions but the accordion title is not clickable.

I changed the them back to Garland but this did not solve the problem.

davidburns’s picture

davidburns
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Closed (fixed)

newest dev branch should fix this problem.

adding a couple new features and releasing another beta branch before final release. sorry for the delay