SuluFormBundle icon indicating copy to clipboard operation
SuluFormBundle copied to clipboard

Template improvements

Open plozmun opened this issue 3 years ago • 1 comments

Q A
Bug? no
New Feature? Yes
Bundle Version -
Sulu Version 3.0?
Browser Version All

Actual Behavior

Currently the templates of both the fields and the general form are defined by default and are not overwritable. It's possible to override some services but it would be nice to make it simple.

Expected Behavior

Some suggestions to improve the bundle:

  • Allow to override main template to remove or add some custom fields ( remove email section or add new section to integrate with CRM`s or similar )
  • Templates: It would be nice to create templates for forms. Example: Two columns forms , step forms, etc..

plozmun avatar Sep 01 '21 09:09 plozmun

Thanks for creating this issue.

Allow to override main template to remove or add some custom fields ( remove email section or add new section to integrate with CRM`s or similar )

This is something I already have in mind that Email part should be optional. Also the email configuration should not be in the same tab as the form configuration. For this already https://github.com/sulu/SuluFormBundle/issues/32 was created.

Templates: It would be nice to create templates for forms. Example: Two columns forms , step forms, etc..

I'm not sure if I want to implement this things. If a Form is shown as step by step form or as grid form depends on the design or theme. In sulu we always try to separate the content from its presentation. I'm not sure yet if we would add something like that, as a Theme A could show the form as step by step form and Theme B as normal column form. So the implemented theme itself would and should have the control how the form is shown and the form bundle itself should not care about it. Still it should be able that the form entity is extendable with custom fields by overwrite the form_details.xml with new fields and by decorating the FormManager service.

alexander-schranz avatar Sep 13 '21 15:09 alexander-schranz