Currently, a collapsible div has a class 'collapsed' when it's initially collapsed, but that class persists when the div is expanded. Likewise, a collapsible div doesn't get the collapsed class when it's collapsed. This patch toggles the collapsed class when collapsing/expanding a collapsible div. This can be very useful for themers.

CommentFileSizeAuthor
collapse_div_toggle_class.patch419 bytesmarcvangend
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Stalski’s picture

Stalski CreditAttribution: Stalski commented
Status: Needs review » Fixed

Correct, forgot about that one. Pushed to git. Thx for the patch.

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
CreditAttribution: marcvangend commented

Thanks for committing!

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

I don't believe this is what was intended, but the file collapse_div_toggle_class.patch was added to the module's codebase itself, in addition to the patch being applied.

marcvangend’s picture

marcvangend
he/him
Primary language Dutch
Location Amsterdam
CreditAttribution: marcvangend commented
Status: Fixed » Needs work

I'm always honored when people commit my code :-)

Stalski’s picture

Stalski CreditAttribution: Stalski commented

@DamienMcKenna, sorry that was not intended. Sorry for that.

Stalski’s picture

Stalski CreditAttribution: Stalski commented
Status: Needs work » Fixed

Patch file removed and it was applied already.

Status: Fixed » Closed (fixed)

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