This is a continuation of #1148808: Webform missing advanced components (add support for date component) (which added support for dates), to add support for the grid component to the Form Builder Integration.

CommentFileSizeAuthor
#3 form_builder_grid-d6.patch7.69 KBquicksketch
#3 form_builder_grid-d7.patch8.73 KBquicksketch
#2 form_builder_grid-d6.patch7.7 KBquicksketch
#2 form_builder_grid-d7.patch8.74 KBquicksketch
#1 webform_grid_process-d6.patch4.48 KBquicksketch
#1 webform_grid_process-d7.patch4.34 KBquicksketch

Comments

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Needs review
StatusFileSize
new webform_grid_process-d7.patch4.34 KB
new webform_grid_process-d6.patch4.48 KB

Well considering the Grid component is the most complicated component in Webform, this wasn't *too* bad. THe grid component needed some reworking in the Webform queue (see #1151714: Grid component should use a #process function to expand itself) but then things went pretty smoothly. Patch attached.

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new form_builder_grid-d7.patch8.74 KB
new form_builder_grid-d6.patch7.7 KB

Sorry those were the patches for Webform module, not Form Builder. Here we are.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed
StatusFileSize
new form_builder_grid-d7.patch8.73 KB
new form_builder_grid-d6.patch7.69 KB

I found one small issue in review that made it so that the "Use custom keys" checkboxes were not being maintained between edited (though they were saving properly). Final patches, committed to both branches.

Status: Fixed » Closed (fixed)

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