[regression] dijit.form.TextBox - type=hidden, shows a border

[Pete Smith]

I see the css has been overhauled. I have :

new dijit.form.TextBox({name: this.name, type: 'hidden', value: this.value});

Which shows up with a gray border on it. This is coming from Common.css on line 22. Hidden should auto apply a hidden class should it not?

This did not do this in 1.4.X.

[bill]

Maybe from [21560]?

[Kitson Kelly]

I am experiencing this too... I have attached an example attached that demonstrates this. I get the following in the rendered HTML for the field:

In 1.4:

<input type="hidden" name="postAction" autocomplete="off" dojoattachevent="onmouseenter:_onMouse,onmouseleave:_onMouse" dojoattachpoint="textbox,focusNode" class="dijit dijitReset dijitLeft dijitTextBox" value="search" id="dijit_form_TextBox_0" tabindex="0" widgetid="dijit_form_TextBox_0">

In 1.5.0b2:

<div wairole="presentation" id="widget_dijit_form_TextBox_0" class="dijit dijitReset dijitInline dijitLeft dijitTextBox" role="presentation" dir="ltr" widgetid="dijit_form_TextBox_0"><div class="dijitReset dijitInputField dijitInputContainer"><input type="hidden" name="postAction" autocomplete="off" dojoattachpoint="textbox,focusNode" class="dijitReset dijitInputInner" value="search" id="dijit_form_TextBox_0" tabindex="0"></div></div>

[Kitson Kelly]

Provides an example of a form where the hidden field gets rendered.

[Kitson Kelly]

I took a look at the code, and it does seem to be related to [21560], because prior to it, the TextBox? template simply had a <input> tag and now it has the two <div> that wrap the <input>.

Personally, I would submit a patch, but I am not sure how it should be fixed.

My suggestion would be to fix it more fundamentally at the dijit.form._FormWidget to apply a dojo.style to the .domNode whenever the type="hidden", but I don't know enough to say if that would break anything else.

[Kitson Kelly]

I did a quick fix based on my suggestion above and have supplied a patch. It seems to work for me in several different scenarios and doesn't have any ill affects with other Widgets.

My CCLA has been on file from 28 April with Aimee.

[Kitson Kelly]

Does a dojo.style(this.domNode,"display","none") when the type="hidden"

[Douglas Hays]

CSS only solution for comparison to enable custom styling based on type

[Douglas Hays]

I attached an alternate non-javascript patch (textbox subclasses would need the new class added as well) for Bill to choose from.

[Douglas Hays]

(In [22145]) Fixes #11049, #11058. For TextBox? widgets of type file|hidden, swap the multi-node template for a single INPUT version since styling capabilities are only minimally required/allowed. Added the ability to select the single node template (for backward compatibility) by setting templateString="input".