I've followed the README, but I can't seem to get this to work at all. Here's what it put in the header on my site:

<script type="text/javascript" src="/misc/jquery.js?v"></script>
<script type="text/javascript" src="/misc/drupal.js?v"></script>
<script type="text/javascript" src="/modules/jstools/jstools.js?v"></script>
<script type="text/javascript" src="/modules/collapsiblock/collapsiblock.js?v"></script>
<script type="text/javascript" src="/modules/jstools/jquery.cookie.js?v"></script>
<script type="text/javascript">jQuery.extend(Drupal.settings, { "basePath": "/", "jstools": { "cleanurls": true, "basePath": "/" }, "collapsiblock": { "blocks": { "block-block-3": "3" }, "default_state": "1" }, "titleSelector": "h3" });</script>

That all looks correct. It's got the right block - #block-block-3 - and it's using ngp.inc to get the H3 titleSelector, but for whatever reason it isn't actually doing anything on the page, and no errors are generated:

http://ngpsoftware.ngphost.com/

Any ideas?

CommentFileSizeAuthor
#2 collapsiblock.module.patch1.08 KBseanr
#1 collapsiblock.js_.txt1.79 KBseanr

Comments

seanr’s picture

Status: Active » Needs review
StatusFileSize
new1.79 KB

Found the culprit. It's not respecting the titleSelector because it's not being output under collapsiblock in the settings. This patch to collapsiblock.js fixes it, but you might also want to fix it on the other end (collapsiblock_load - looked at that one but wasn't sure how to fix it).

seanr’s picture

StatusFileSize
new1.08 KB

Got it. This fixes it from the other end instead. Probably better since it keeps everything grouped inside collapsiblock, but take your pick. ;-)

nedjo’s picture

Status: Needs review » Reviewed & tested by the community

Thanks, good catch, looks good.

nedjo’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, applied.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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