Missing "Layout Container" form component in add a field area within the Webform.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | Screenshot from 2015-12-08 17:28:42.png | 11.36 KB | colemanw |
| #9 | Image 2015-12-07 at 5.10.08 p.m..png | 23.77 KB | rv0 |
| #8 | webform_layout_form_builder-1721508-8.patch | 3.69 KB | rv0 |
| #6 | webform-layout-7.x-2.2-form-builder.diff | 3.69 KB | Rajan M |
Comments
Comment #1
colemanw commentedThis module is not yet compatible with form_builder.
Adding it may not be entirely straightforward because a layout box is not a field but a container. I have not looked into how the js for this would work.
Comment #2
liza commentedHI! For the sake of helping other front-end developers avoid wasting time thinking theyve done something wrong upon installing the module and not seeing it anywhere on the UI, could you please state that clearly and in bold letters on the project description? Thanks!
Comment #3
colemanw commentedOk done.
Comment #4
danchadwick commentedIf this module were compatible with Form Builder, I would consider rolling it into webform core. If this is desired, please open an issue in the webform issue queue.
This module looks quite useful to me.
Dan Chadwick
webform co-maintainer
Comment #5
colemanw commentedThanks Dan :)
I agree, it would be great to get this compatible with form_builder and rolled into core.
I'll see what I can do...
Comment #6
Rajan M commentedCurrently form_builder module provide dropable interface within 'fieldset' component and seems like code is tidely coupled with the 'fieldset'.
IMO simple way to make 'Webform Layout work to with form builder', which will allow us to add element within its region, is to make 'Layout box' behave like fieldset while building a form.
I tried this and looks like it works fine. I am attaching a patch for form builder 7.x-2.2, hope this will help to got it working :)
This patch is part of development work by Coworks.
Comment #7
colemanw commentedThat's clever.
One change - the module_exists() call should check for form_builder_webform not form_builder.
Comment #8
rv0 commentedI just tested and reviewed Rajan's patch and it's working great for me
I took the liberty to re-roll the patch using colemanw's suggestion from #7 (module_exists changed to form_builder_webform)
Comment #9
rv0 commentedCould use an icon though:
Comment #10
Rajan M commentedCool Thanks!
Comment #12
colemanw commentedCommitted the patch and added an icon.
I wouldn't call this level of integration with form_builder "awesome"; it doesn't give a very wysiwyg experience since a layout box generally makes fields align side-by-side and they don't appear that way in the form_builder UI.
But it's a good step forward and at least now the two modules don't break each other.
Comment #13
colemanw commentedComment #14
rv0 commented@colemanw: cool
Yeah, a more wysiwyg experience would be cool.
Thanks for the attribution, although all credit should go to @Rajan M, I merely changed one line as per your suggestion ;)
Comment #16
rv0 commentedThis should be removed from project page, or at least mentioned it works in dev.
Comment #17
colemanw commentedDone.
Comment #18
thronedigital commentedSeems to no longer be working again?
Comment #19
colemanw commented@thronedigital
Details?
Comment #20
adamtong commentedit is not working with latest form builder version