Enable text selection in Internet Explorer

[stakach]

Currently (Unlike all other browsers) there is no way to select text in draggable objects in internet explorer even if a handle is provided.

This is caused by two pieces of code.

Firstly: (This code should be removed)

in $.ui.mouse -> mouseInit
// Prevent text selection in IE
if ($.browser.msie) {
	this._mouseUnselectable = this.element.attr('unselectable');
	this.element.attr('unselectable', 'on');
}

Secondly: (These functions require the last line of code to be added)

in $.fn.extend

enableSelection: function() {
	return this
		.attr('unselectable', 'off')	//Not sure if needed now
		.css('MozUserSelect', '')
		.unbind('selectstart.ui');
		.unbind('selectstart.disableTextSelect');	//ie select enable
},

disableSelection: function() {
	return this
		.attr('unselectable', 'on')	//Not sure if needed now
		.css('MozUserSelect', 'none')
		.bind('selectstart.ui', function() { return false; })
		.bind('selectstart.disableTextSelect', function() { return false; }); //ie select disable
},

What do people think? Works for me.

[stakach]

Tested in Firefox 3.5, IE 6-7-8, Chrome, Safari 4 for windows with no issues

[stakach]

IE selection patch

[Jörn Zaefferer]

Looks good.

[stakach]

Thanks. I'm using this patch with a web application for sydney university and everything is working perfectly.

Also there are no syntax errors in the patch unlike my description. :)

[Jörn Zaefferer]

I don't see why you bind the selectstart event twice. Though removing the $.browser.msie check from mouseInit definitely makes a difference. I have no idea why that is there anyway.

[stakach]

Yeah - I'm using selectables, resizables and draggables without problems. Second bind is not needed.

This is my first jQuery project, been using script.aculo.us, and wasn't sure about event namespaces.

[Scott González]

Fixed in r3546. Thanks stakach.