Closed (fixed)
Project:
Collapsiblock
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Mar 2010 at 16:17 UTC
Updated:
28 Aug 2010 at 20:37 UTC
The instructions say to create a file anemd [themename].inc to define the custom selector for the collapsiblock link if it is not an H2 tag. I tried that, but the file never gets loaded and I cannot find anything in collapsiblock.module that would appear to do that. As a result the blocks will not expand and collapse. I was able to work around it on my site by adding a field to the admin settings page and using a variable_get in collapsiblock_load to add it to the javascript, but obviously that won't be theme specific. It'd be nice to get this working back the way it was intended (and even nicer to just have the function in template.php - no idea why it needs to be in a new .inc file).
Comments
Comment #1
seanrFigured out what's going on - it's looking for [themename].inc in /sites/all/modules/collapsiblock/theme/. The trouble with that is that any file added there will get erased when performing an update with drush. The module really needs to look in the theme's directory so that drush updates don't break it.
Comment #2
seanrJust got bit by this one again. Has anyone other than me read this post? ;-)
Comment #3
nedjoYes, this should be fixed. I'd welcome a patch.
Comment #4
gagarine commented#887074: Using a more flexible JS selector for block title fix this issue. [themename].inc is not any more needed. Please take a look at #887188: Provide the ability to set settings in theme's configuration page
Comment #5
gagarine commented