accept closure attributes (eg wrapper.class) #698
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change accepts attributes to be closures (resolved by Laravel's
value()
). That's not useful for most fields, because all context is present where the field is defined. But it is useful forcollection
fields, because you define the same attributes for all the items/child forms. This way you can dynamically add different attributes for different items:I use this to highlight some collection items if they contain problems.
$field
is the collection item field. In this case aChildFormType
becausetype = form
. It could be aInputType
, which doesn't have agetForm()
and doesn't really know anything useful. It's most useful for child forms.