Ability to Add Labels to each Row (not fillable textfields)

Hi kyleheney ,

Yes I can. But this will not a part of webform_matrix_component. If you require I can made this for you.

yes something like that or where we can configure column as well as row.

If you wanna fixed this mail me chetansinghal1988@gmail.com

I have the same need to have separate labels for each row. I have tried to find a solution for this together with the webform component but havent had any success. I tried the webform table module but it didnt work at all with the newest release of webform.

I have attached an image that you can take a look at.

This is enhancement I will add this my next release.

Very good, let me know when you have added the row feature in the next release. I can test the feature for you an d give feedback.

Very good, let me know when you have added the row feature in the next release. I can test the feature for you an d give feedback.

I am not planning next few days. If you require this feature let me know.

This feature seems like a must have for a module like this. I'd love to see it as well. I'll start looking at the code and If I come up with something useful will submit a patch.

Yes I think also its a must for this module. Looking forward to this feature.

Subscribe

I've written a patch, which allows different labels on each row. When the 'label' component is selected in the matrix component edit screen, there will be one textfield per row where the label can be added.

The module maintainer should probably review the following hunk in particular:

@@ -323,7 +333,10 @@ function theme_webform_matrix_table($variables) {
   $output = "<div class='form-item webform-component webform-component-matrix webform-component--$form_class webform-component-$form_class' id='$id'>";
   $output .=!empty($element['#matrix_header']) ? "<label>" . $element['#matrix_header'] . "</label>" : "";
   $output .= theme('table', array('header' => $headers, 'rows' => $rows, 'attributes' => array('id' => array("$id-table"))));
-  $output .= drupal_render_children($element);
+
+  // Render the 'add row' button.  Add row button id is in format add_row_formId.
+  $add_row_id = 'add_row_' . $element['#webform_component']['form_key'];
+  $output .= drupal_render_children($element, array($add_row_id));

I've made an assumption that the drupal_render_children($element) call is for the purpose of rendering the 'add row' button. Is this correct? The problem I had was that all labels were being rendered for a second time below the table, so I changed the drupal_render_children call to render ONLY the 'add row' button.

New patch. Removed d.o. packaging info, which was accidentally committed.

Backport to 3.x

The patch fails on the .info file. But it works, thanks!
I'm using 7.x-4.11.

Thanks to all of you and Wish you very happy new year.

I will review your patches and requested functionality.

Any news on this? I'm using this on a webform that will go live soon. It works for me.

Hi littledynamo,

Your patch is working as per you implementation but i have some issue.
Can you please fixed the following issue which are in the file addLevel2.png

@devendra.yadav Does that happen when clicking the 'Add row' button?

Yes,

I think the following solution.
1- Your added label should not be mandatory.
2- Pick the Label of Header value when we are pressing the Add row button.
3- Put a Note in description regarding the same cause.

Thanks.
Devendra

New patch attached.

Fresh patch. Added isset check to suppress a warning.

The patch from #34 works great for me. Thanks, everyone!

Yes the patch #34 is working perfect. We really needed this patch! Thanks!

Yes the patch #34 is working perfect. We really needed this patch! Thank you so much!

webform_matrix_component-7.x-4.x-dev. Patch #34. I get 10 warnings (to 10 rows of the matrix):
Notice: Undefined index: #type в функции theme_webform_matrix_table() (строка 397 в файле /sites/www/molbank/test.molbank.ru/sites/all/modules/webform_matrix_component/webform_matrix_component.module).

Patch #34 worked great for me

Patch #34 worked great at adding the labels for each row option for me as well (on 7.x-4.16)...thanks! However as noted from TLTHades on comment #38 I also received this warning once the patch was installed:
Notice: Undefined index: #type in theme_webform_matrix_table() (line 397 of sites\all\modules\webform_matrix_component\webform_matrix_component.module).

I see these warnings when clicking the Next/Previous button and they do match up with how many rows are in the matrix.

Thanks littledynamo,

To develop this functionality. I review and verify the same functionality. it is working fine but getting some warnings when i click on button Add a row.

Notice: Undefined index: #type in theme_webform_matrix_table() (line 397 of /var/www/d7/sites/all/modules/contrib/webform_matrix_component/webform_matrix_component.module).
    Notice: Undefined index: #type in theme_webform_matrix_table() (line 397 of /var/www/d7/sites/all/modules/contrib/webform_matrix_component/webform_matrix_component.module).
    Notice: Undefined index: #type in theme_webform_matrix_table() (line 397 of /var/www/d7/sites/all/modules/contrib/webform_matrix_component/webform_matrix_component.module).

Please look that those.

Updating the patch with code style fixes and added an additional check for the #type attribute. The warning/notice goes away on my local test.

Fixed in dev version.

hi ...

patch #34 and patch #43 don't work with me.
i use Webform Matrix Component (7.x-4.17)
and i applied patch manually because my site is on shared server!

any idea?

hi

7.x-4.x-dev (2015-Sep-14 ) work correctly.