The issues fixed in the 7.x-1.5 release (https://www.drupal.org/node/2173937 in particular) change the options given to the accordion from using `autoHeight` to `heightStyle` to provide the desired variable-height setting for users with a newer version of jQuery. This setting was changed in jQuery UI 1.9, but Drupal's default is 1.8.

To maintain backward compatibility, both options should be specified or jQuery Update should become a dependency of this module to help push users forward. I would vote for the first, of course.

Patch to come.

Comments

awochna created an issue. See original summary.

awochna’s picture

StatusFileSize
new465 bytes

This fixed the issue in my sites.

I have added the `autoHeight` option and left the `heightStyle`. Extra options simply get ignored, so it should cause no problems.

rschwab’s picture

Status: Active » Reviewed & tested by the community

The patch here worked for me on 7x-1.5, although it should probably be moved to #2173937: Accordion Element has fixed height in HTML and this issue closed as a duplicate.

benjifisher’s picture

+1 for RTBC.

@awochna: thanks for the fix.
@rschwab: Since #2173937: Accordion Element has fixed height in HTML is already committed (and released as part of 7.x-1.5) I think it is appropriate to have this new issue and post the patch here.

sokru’s picture

+1 for commiting this to dev branch.

wipeout_dude’s picture

I'll add my +1.. Added the patch to my site running 1.5 and solved the issue..

timdavison’s picture

+1 from me too. Worked great for site running 1.5

jeebsuk’s picture

Another +1 here with 1.5. Looks to resolve the issue nicely.

nils.destoop’s picture

Status: Reviewed & tested by the community » Fixed

Thx for the patch. I committed it to dev

  • nils.destoop committed 825da1b on 7.x-1.x authored by awochna
    Issue #2649074 by awochna, benjifisher, rschwab, sokru, wipeout_dude,...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.