Detached DOM tree entry

[tchatziarx]

Regarding some old bug reports like ​https://bugs.jqueryui.com/ticket/10629 and ​https://bugs.jqueryui.com/ticket/9555.

I was playing around with jquery-ui and I found this Detached DOM tree entry without using the datepicker from jquery-ui (which by the way does not appear if you call for example $("#foo").datepicker().

I also made a small change to the code in order to fix this and here it is in case you would want to include it in some newer version.

jquery-ui-1.12-1.js on line 7305 I commented the following line

//~ this.dpDiv = datepicker_bindHover( $( "<div id='" + this._mainDivId + "' class='ui-datepicker ui-widget ui-widget-content ui-helper-clearfix ui-corner-all'></div>" ) );

and placed the same line a bit changed, as far as "this." concerns, after line 9241

/* Append datepicker main container to body if not exist. */
if ( $( "#" + $.datepicker._mainDivId ).length === 0 ) {
	$.datepicker.dpDiv = datepicker_bindHover( $( "<div id='" + $.datepicker._mainDivId + "' class='ui-datepicker ui-widget ui-widget-content ui-helper-clearfix ui-corner-all'></div>" ) );
	$( "body" ).append( $.datepicker.dpDiv );
}

[Scott González]

I'm not sure what this ticket is for. Are you just trying to report that you've also encountered the previously reported bugs?

[tchatziarx]

Yes I am reporting that I've also encountered the previously reported bugs (just by including jquery-ui, without using any of its wigdets) and I also made the small change I mention in order to fix it.

[Scott González]

Duplicate of #9555.
Ok, so I'll close this as a duplicate. The 1.1.3.0 release will include a complete rewrite of the datepicker widget, so this change won't apply to the new code.