Form Field: Add State modal window Date field does not work

Your last sentence is extremely helpful. The difference between the 2 is this:

"Form field: set disabled" uses $element = &$this->getTargetElement() to find the form element to deal with.

"Form field: add state" uses the same statement, followed by $element = &$this->jumpToFirstFieldChild($element) which receives the first field child to which the state flag is then added.

I can see how this may behave different if you have a multi-value field, like date fields with date and time inputs. I wonder if this should be resolved by also supporting state for containers, like e.g. a wrapper around a complex widget?

thank you, it seems you are right.

I have also tested with custom_field → module and it does not hide the field by default when the widget is "Stacked" however when you change it to Flaxbox and use "fieldset" or "details" type of Wrapper it works.

So I wanted to replicate the same as you suggested with the date and for me the fastest way is to use field_group → module, so I added field group HTML element , with "div" element and added extra CSS class "datewrapper" and added CSS like

.datewrapper {
  padding: 10px;
  border: 1px solid #ccc;
  background-color: #f9f9f9;
}

However I can still see the date field , of course the css is applied correctly, but still not working with visibility. I have tried also with fieldset type of field group with the same result :(

Any suggestion? Any advice would be greatly appreciated.

It works now..yupii.

Instead of field group i used ECA directly.

Form:add grouping element and I have added element name like "datex" , title and fields just field_date so I was able to use element name "datex" as my field name of Add state.

However as Title is required i was not happy with details wrapper so I added additional "form: add container element" and element name "datex" so now everything works as expected.

However I am not sure how this "form: add container element" works.
I think I do not need both forms at the same time but currently this is my workaround.

Do you think there can be any patch created which will add container to date fields by default so no need to do such "workarounds"?