Re: Quickform input=reset element display bug?

Hi! Mike Carter wrote:

Well, I'm pretty sure Array renderer *does not* add : to labels. You probably have :'s in your template (which you didn't show).

I'm using the standard Dynamic Smarty template which was distributed with the current Quickform. I haven't actually touched the template code - I just borrowed it from html/docs/renderers/template/smarty-dynamic.tpl

Clarification: that's not 'standard' template, that's 'example' template. ;]

Could it be this template is not taking into account the possibility of some elements not having a matching label? Well, looks like it does not take into account the possibility of reset buttons, because of this:

            <!-- submit button (don't display on frozen forms) -->
            {elseif $element.type eq "submit"}
                {if not $form.frozen}
                <tr>
                    <td>&nbsp;</td>
                    <td align="left">{$element.html}</td>
                </tr>
                {/if}

I *think* it should be

            {elseif $element.type eq "submit" or $element.type eq "reset"}

I am not that familiar with Smarty, so please test this or try to ask Thomas Schulz, who authored the Smarty examples.