Closed (fixed)
Project:
Collapse Text
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
14 Jan 2011 at 13:25 UTC
Updated:
30 Mar 2011 at 16:01 UTC
Any word on when this might be available for D7? This is a great module we use in D6 and we are anxious to migrate. Thanks for all of your efforts!
Comments
Comment #1
Roulion commentedsubscribe
Comment #2
keha3912 commentedsubscribe
Comment #3
koppie commentedSubscribing!
Comment #4
pukku commentedHi! I'm hoping to start looking at Drupal 7 in the next few weeks. As this module is relatively simple (well, mostly), this will hopefully be one of the first I work on...
Ricky
Comment #5
d.jankovics commentedsubscribe
Comment #6
pukku commentedHi! I've created a first test at a port to D7. There are three things to note.
1. This is my first D7 module.
2. The style and title options now require quotes around their parameters. Double-quotes, actually. I am working to relax this a bit, but my guess is that they will still require some sort of quote from now on. This change may start to affect the D6 version as well, as there is a fatal bug in the parsing regex that I need to figure out.
3. I'm not sure about the cache problem. I'm still trying to figure out how to make sure that the collapse javascript library is loaded. Right now I can't seem to get it not loaded, but I haven't spent a lot of time on it. If anyone can tell me for sure that it is not loaded, and thus the collapsing doesn't happen, that would be great.
Thanks,
Ricky
Comment #7
pukku commentedActually, I just changed the code that caused issue 2 above. The method of handling the tag is now much more robust, and will also try to fix missing quotes. If there is a quote in the title or style option though, things will not work the way you want them to.
Comment #8
pukku commentedPlease test the beta that I've posted (it should show up in a few hours).
Comment #9
pukku commented7.x-2.1 released.