Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#843 closed defect (invalid)

Add widget class to ValidateTextbox template

Reported by: mike@… Owned by: anonymous
Priority: high Milestone:
Component: Widgets Version: 0.2
Keywords: Cc: mike@…
Blocked By: Blocking:

Description

Unlike most Dojo widgets, the ValidateTextbox? (and all derivatives) don't include a class attribute, indicating the widget type.

I haven't submitted a CLA yet, but I'll be doing that first thing in the morning.

Attachments (1)

validate.js.patch (649 bytes) - added by mike@… 13 years ago.
Patch to add a class attribute to the ValidateTextbox? et al <div> tag

Download all attachments as: .zip

Change History (3)

Changed 13 years ago by mike@…

Attachment: validate.js.patch added

Patch to add a class attribute to the ValidateTextbox? et al <div> tag

comment:1 Changed 13 years ago by bill

Cc: mike@… added
Resolution: invalid
Status: newclosed

Hi Mike. You can already set the class by specifying the className="..." attribute. Doesn't that work for you?

comment:2 Changed 13 years ago by Michael Nachbaur <mike@…>

Setting the className attribute sets the class on the <input/> textbox, not the <div> container that wraps it. The way it works without this patch is by placing a blank div around the validated textbox elements (input, error labels, etc), meaning that each ValidatedTextbox? widget takes up its own line (due to <div> having a default "display: block" style).

Using this patch I can override the default CSS styles to customize the display of my form.

Note: See TracTickets for help on using tickets.