Missing "Layout Container" form component in add a field area within the Webform.

Comments

colemanw’s picture

Title: missing Layout Container form component » Compatibility with form_builder module
Category: bug » feature

This 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.

liza’s picture

Issue summary: View changes

HI! 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!

colemanw’s picture

Ok done.

danchadwick’s picture

If 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

colemanw’s picture

Thanks 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...

Rajan M’s picture

StatusFileSize
new3.69 KB

Currently 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.

colemanw’s picture

That's clever.
One change - the module_exists() call should check for form_builder_webform not form_builder.

rv0’s picture

Version: 7.x-1.0 » 7.x-2.2
Status: Active » Needs review
StatusFileSize
new3.69 KB

I 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)

rv0’s picture

StatusFileSize
new23.77 KB

Could use an icon though:

webform builder ui

Rajan M’s picture

Cool Thanks!

  • colemanw committed 7ac0c31 on 7.x-2.x authored by rv0
    Issue #1721508 by rv0, Rajan M: Compatibility with form_builder module
    
  • colemanw committed 851f70e on 7.x-2.x
    Issue #1721508 Add icon for form_builder
    
colemanw’s picture

Committed 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.

colemanw’s picture

StatusFileSize
new11.36 KB

rv0’s picture

Status: Needs review » Fixed

@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 ;)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

rv0’s picture

Status: Closed (fixed) » Active

This module is not yet compatible with the form_builder module.

This should be removed from project page, or at least mentioned it works in dev.

colemanw’s picture

Status: Active » Closed (fixed)

Done.

thronedigital’s picture

Seems to no longer be working again?

colemanw’s picture

@thronedigital
Details?

adamtong’s picture

it is not working with latest form builder version