Accordion title not clickable & not expanding on click

ckng - January 23, 2008 - 07:10
Project:Panels Accordion
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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>';

#1

Veggieryan - February 19, 2008 - 11:19

+1 for this.

works perfectly for me in panels 2 beta 2.

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

#2

davexoxide - May 10, 2008 - 19:13

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.

#3

davexoxide - May 10, 2008 - 19:13

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

#4

apratt - June 6, 2008 - 19:32

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...

#5

apratt - June 10, 2008 - 16:16

Updating to panels beta5 seemed to solve the problem.

Angus

#6

leo.ruffini - August 29, 2008 - 09:11

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.

#7

davexoxide - November 29, 2008 - 21:53
Status:patch (code needs review)» closed

newest dev branch should fix this problem.

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

 
 

Drupal is a registered trademark of Dries Buytaert.