"Slight Performance Tweaks" is a severe breakage

[rotdrop]

Referring to commit

d2564a0

This commit damages consistency in the calling code (attached event handlers references to the this.$input element) because it deletes the current $input element and attaches another similar element again. But similar in the context of object references is just not "equal" but "different". This commit is altogether evil.

[rotdrop]

@risadams Can you please revert this commit? It breaks the consistency with jQuery objects in the calling code as it deletes DOM elements and replaces them by other similar DOM elements.

[rotdrop]

Talking about these lines of code:

selectize.js/src/selectize.js

Lines 268 to 278 in e6ca6d3

	var inputPlaceholder = $('<div></div>');
	var inputChildren = $input.children().detach();
	$input.replaceWith(inputPlaceholder);
	inputPlaceholder.replaceWith($input);
	this.revertSettings = {
	$children : inputChildren,
	tabindex : $input.attr('tabindex')
	};

This code replaces the primary DOM-element by an equivalent copy. This is at the very least extremely unexpected.