-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/form section #600
Feature/form section #600
Conversation
…eature/accordion
add js action to accordion
…eature/accordion
Change Item to Section .
…eature/accordion
Use Accordion form layout for thighter integrarion with accordion.
form accordion layout now use them to customize error activity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just minor changes, please address, then good to merge!
$f->setModel($m, false); | ||
|
||
$sub_layout = $f->layout->addSubLayout(); | ||
$sub_layout->setModel($m, ['first_name', 'last_name']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can $m be optional here btw?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also what happens if a different model is used here... ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Theoretically $m can be optional and then sub-layout could take model from it's parent layout ...
About different model - no idea, ave not tested that :)
Note
Replace previous Feature/Accordion.
This PR add ability to setup a layout section in Form where field can be added to the view.