-  
 -   Notifications  
You must be signed in to change notification settings  - Fork 388
 
Add ux collection type component #397
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
base: 2.x
Are you sure you want to change the base?
Add ux collection type component #397
Conversation
From UXCollectionType to the view
Co-authored-by: jmsche <contact@jmsche.fr>
Co-authored-by: jmsche <contact@jmsche.fr>
Co-authored-by: jmsche <contact@jmsche.fr>
Co-authored-by: jmsche <contact@jmsche.fr>
Co-authored-by: jmsche <contact@jmsche.fr>
Co-authored-by: jmsche <contact@jmsche.fr>
Co-authored-by: Romain Monteil <monteil.romain@gmail.com>
Co-authored-by: Romain Monteil <monteil.romain@gmail.com>
Co-authored-by: Romain Monteil <monteil.romain@gmail.com>
Co-authored-by: Romain Monteil <monteil.romain@gmail.com>
| class UXCollectionEntryToolbarType extends AbstractType | ||
| { | 
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.
The UXCollectionEntryToolbarType and UXCollectionToolbarType allows to add additional buttons later via a form extension. Example can be a Copy & Paste Functionality.
|   Does this approach support nested dynamic collections (here is a test case: https://github.com/symfony/ux/tree/e901026aa1d2e2cd2b27a5796962bc8fc4b5dcca/src/Collection/Tests/app/Form)? For the record, I proposed an alternative patch in #398 which doesn't require changing the existing forms to use this feature (see #398 (comment) for the rationale).  |  
|   @dunglas yes you can have and I'm using nested collections this way. But like documented in the   |  
This PR is based on the awesome work of @stakovicz in: #90
TODO
data-controllerfor customizationentry_typeinstead ofux_entry_type