Hi!

upon trying theme a webform using salamander skin template http://drupal.org/project/salamanderskins by following the instructions on:
http://1013.fi/cms/theming-drupal-web-forms , the problem occured:

Unnecessary form table rows were added to the webform:

/td><td></td> </tr>
 <tr class="even"><td class="label_cell"><div class="form-item">
 

</div>
</td><td></td> </tr>
 <tr class="odd"><td class="label_cell"><div class="form-item">

how to remove these odd even rows???
thank you

Comments

Maine’s picture

Maine commented
Title: Themeing Drupal 6 Webforms » Theming Drupal 6 Webforms
Priority: Critical » Minor
Status: Active » Postponed (maintainer needs more info)

Hi curiousEngine,

I wrote that piece at 1013 about Theming Drupal Webforms, and I'm more than happy to help if there are any bugs left. However, with this information there's nothing I can do. Please, 1) explain what's the actual problem, 2) post the whole page source and 3) post the webform structure.

Regards,
-Juho

curiousEngine’s picture

curiousEngine commented

Hi Juho!
Nice to receive a reply on this theming issue.

In fact the php scripts work to put the fields in a table.
However:
The problem is maybe with zebra in drupal theme. There are additional form-items which appear in the webform.

curiousEngine’s picture

curiousEngine commented
Category: support » bug
Priority: Minor » Critical
Status: Postponed (maintainer needs more info) » Active

Hi Juho!

In fact its when u introduce a Page Break in the webform, additional table rows are created...

can this bug be solved pleased?

thanks

jacine’s picture

jacine
she/her
Location New York
commented
Project: Drupal core » Webform
Version: 6.x-dev »
Component: theme system » Miscellaneous

Sounds like this belongs in the webform project queue, not Drupal. Please update this issue to the version you are referring to.

quicksketch’s picture

quicksketch
he/him
commented
Category: bug » support
Priority: Critical » Normal

Support for theming is not provided in the Webform queue (at least by maintainers). If left open for more than a few days here without response I'll close it directly.

Maine’s picture

Maine commented
Status: Active » Closed (fixed)

Not Drupal/Webform issue. Problem originated from a code snippet in my article that didn't handle page breaks properly. I updated the code to skip over type-less form elements, as typelessness seems to indicate item is not on the current page.

curiousEngine’s picture

curiousEngine commented

Ok
Thank you Juho for this quick fix!

I have checked the updated code on: http://1013.fi/drupal/sites/default/files/file/theming-drupal-forms/form...
and applied it.

It works!

Thanks for the support..

Keep up the good job :)

Regards,

http://ashesh.ramjeeawon.info