Opened 13 years ago

Closed 12 years ago

#2650 closed defect (fixed)

HTML generated by DatePicker doesn't include an ID

Reported by: joao_g_fonseca@… Owned by: Adam Peller
Priority: high Milestone: 0.9
Component: Dijit Version: 0.4.2
Keywords: DatePicker Cc: joao_g_fonseca@…
Blocked By: Blocking:

Description

When using the DatePicker? widget:

<div dojotype="DatePicker?" widgetId="x"></div>

The corresponding HTML doesn't keep the ID in the container. It should expand to something like:

<div id="x" class="datePickerContainer" ...

With the current way, the widget is hard to position using CSS.

Change History (5)

comment:1 Changed 13 years ago by tw@…

It should work to add

id="${this.widgetId}"

to the opening div tag in the templateString of src/widget/DatePicker.js

BTW, up to 0.4.2rc2, the template for DatePicker? was specified via templatePath but in the final ajax build I downloaded, it's included as templateString in DatePicker?.js. Is there a special reason for this change?

comment:2 Changed 13 years ago by James Burke

tw@…: For the final 0.4.2 build, all widget templates where interned via the intern-strings build target, to improve load time of the widgets. If you prefer not to have that, then you can download the source for 0.4.2 and create a build that does not use the intern-strings target.

comment:3 Changed 12 years ago by bill

Component: WidgetsDijit
Milestone: 0.9
Owner: changed from bill to Adam Peller

We don't use widgetId anymore (we just use id). Adam?

comment:4 Changed 12 years ago by Adam Peller

xref #3058

comment:5 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: newclosed

the new form elements in 0.9 preserve id properly.

Note: See TracTickets for help on using tickets.