File changed is lib/pear/HTML/QuickForm/Renderer/Tableless.php

Thanks for the patch.

In general we have avoided modifying the pear code itself and confined our modifications to classes that subclass the pear classes.

I attach another patch which modifies lib/formslib.php instead of the pear files.

But still there is a problem with this patch. I don't know why this is being done :

+ $requiredNote = $form->getRequiredNote();
+ // replace default required note by DOM/XHTML optimized note
+ if ($requiredNote == '<span style="font-size:80%; color:#ff0000;">*</span><span style="font-size:80%;"> denotes required field</span>') { + $requiredNote = '<span class="required">*</span> denotes required field'; + }

If you want to change the default html for the required note we should modify the form class in lib/formslib.php where it is set in the constructor.

The reason for the HTML mangling is simply that that's what the pear code does. Take a look at function finishForm in the Tableless.php file. James' patch is just a direct copy. Though why they want to do that is a mystery!