Attached patch reworks rendering to use theme_table instead of custom theming functions. This lets built in striping, sticky headers, etc to work on webform table elements.

Thanks for the great module!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jelle_S’s picture

Status: Needs review » Needs work
+++ b/webform_table_element.moduleundefined
@@ -73,6 +73,7 @@ function _table_element_expand($element) {
+  $column_titles_complete = false;

what does this do? I see no "if" further down the patch?

attiks’s picture

+++ b/webform_table_element.infoundefined
@@ -4,4 +4,10 @@ description = Table element for the webform module.
+; Information added by drupal.org packaging script on 2011-10-30
+version = "7.x-1.1"
+core = "7.x"
+project = "webform_table_element"

Remove those line from the patch please

+++ b/webform_table_element_formbuilder/webform_table_element_formbuilder.infoundefined
@@ -6,4 +6,10 @@ package = Webform
+dependencies[] = webform_table_element
+; Information added by drupal.org packaging script on 2011-10-30
+version = "7.x-1.1"
+core = "7.x"
+project = "webform_table_element"
+datestamp = "1319966736"

same here

fearlsgroove’s picture

Cruft removed. The variable you mentioned was left over from an abandoned approach.

fearlsgroove’s picture

Status: Needs work » Needs review

nr

attiks’s picture

Assigned: Unassigned » Jelle_S
Jelle_S’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev

Committed to 7.x-1.x-dev, will backport to 6.x
Thanks for the patch! :-)

Jelle_S’s picture

Status: Needs review » Fixed

Committed to 6.x-1.x-dev

Status: Fixed » Closed (fixed)

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