When hiding all fields in a fieldset, the fieldset remains, but is empty. it should be hidden too.

Subscribe !

+1 for that too. It would be really important to see it work.

Thanks!

Edit: or is it a duplicate of https://drupal.org/node/1161314 ?

This is not a duplicate, the other issue is about fieldgroup and not fieldset

I want to hide an empty fieldset

Anyone find a solution for this?

I, too, would like to see this fixed.

Me too!

Same here. Since fieldsets are now built into Drupal, it is important to be able to hide the fieldset, whether we make the fieldset itself conditional or if we hide the fieldset when "empty" (all fields within are conditionally hidden).

Please try this patch that hides fieldsets unless one of their fields is visible.

Great! It works as expected for me.

Well, I guess I was too fast to answer..

This doesn't check recursively (for example it doesn't hide a group which has a "hidden" group which was hidden by this)

Patch #8 works perfectly for me. https://www.drupal.org/node/1161314 would be a more cleaner solution, but for now this will do :)

Patch #8 is a BINGO. Spot on. Thank you anrikun!

Patch works perfectly for the closest fieldset. However, if fieldsets are nested then the patch fails to reach the outer fieldsets.

This also breaks if there are nested fieldsets: The nested fieldsets do not reappear when the condition is triggered.

Any movement on this one? We regularly run into this and just opt to not bother putting things into a fieldset at all because of it...

Changing status (see #13).

On second thought, maybe we should consider this as a duplicate of #1161314: Add basic Field Group support for ANDing conditions (as mentionned in #11).

All right, I've just posted a new version of my patch that should work with nested fieldsets there.
See #1161314: Add basic Field Group support for ANDing conditions

Since #1161314: Fieldgroup support is still not working correctly, I applied this patch and found that it wasn't working for me.

:visible was always returning true, even when the element had display:none.

So I updated it to look for display:none instead.

thanks @anrikun and @roflcopterDorrie ...

@roflcopterDorrie your patch work correctly together with this patch https://www.drupal.org/node/2494385#comment-10708812 on my panopoly distribution 1.46

Already old topic but in my case I had to change this line
var $wrapper = $(e.target).closest('.fieldset-wrapper');
to
var $wrapper = $(e.target).parents('.fieldset-wrapper');
in @anrikun patch for it to work correctly (date with end date was not displaying inside my groupfield)
Hope it can helps someone

I changed last patch by checking if there are any visible children except fieldset description