$(document).bind('state:visible', function(e) {
    if (e.trigger) {
      $(e.target).closest('.form-item, .form-wrapper')[e.value ? 'show' : 'hide']();
    }
  });

Fieldsets have by default:

  1. no IDs
  2. no "form-item" or "form-wrapper" classes

(1) can be solved easily (and I think we should fix that), but I'm not sure what is the way forward for (2).

CommentFileSizeAuthor
#2 drupal.states-types.2.patch2.55 KBsun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

casey’s picture

casey CreditAttribution: casey commented

You can set IDs:

function form_process_fieldset(&$element, &$form_state) {
  ...
  $element['#attributes']['id'] = $element['#id'];
sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Title: Javascript States cannot hide fieldsets » Javascript #states cannot hide fieldsets, radios, checkboxes
Status: Active » Needs review
FileSize
drupal.states-types.2.patch2.55 KB

Same applies to radios and checkboxes.

I hope that attached patch fixes all 3 in 1 shot. At least, I can confirm radios.

rburgundy’s picture

rburgundy CreditAttribution: rburgundy commented

Hello,

I was wondering if the new FAPI: JavaScript States system will allow:
1) hiding a field depending on the value of field a AND field b
2) hiding a field depending on the value of field a OR field b

Thank you

andrewmacpherson’s picture

andrewmacpherson CreditAttribution: andrewmacpherson commented
Issue tags: +FAPI #states

tagging

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented

What's wrong with this patch? RTBC, anyone?

sun’s picture

sun
Primary language German
Location Karlsruhe
CreditAttribution: sun commented
Assigned: Unassigned » sun
moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

this fills a pretty big hole in form element theming. now supports custom attributes and ids. important for skinr as well. Code looks proper and straightforward.

marcingy’s picture

marcingy CreditAttribution: marcingy commented

I can confirm this works for checkboxes as well.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

marcingy’s picture

marcingy CreditAttribution: marcingy commented
Status: Fixed » Reviewed & tested by the community

Just did a CSV up and this doesn't appear to have been committed and no commit exists in the logs for it either.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -FAPI #states

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

Issue tags: +FAPI #states

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.

mattew’s picture

mattew CreditAttribution: mattew commented
Issue summary: View changes

Sorry, maybe I misunderstood, but I encountered the same issue, trying to hide a fieldset (collapsible = TRUE, collapsed = FALSE) with a checkbox, and I found the solution: The fieldset have to explicitly provide an #id attribute.

Maybe this should be pointed out in the documentation?

Thanks