Resizable handles fail to disappear

[rjwalsh@…]

I did some searching and didn't find this anywhere, so here goes. After making an element resizable, then calling

.resizable('disable');

the object is no longer resizable but the resize handles still appear on mouseover. This is really a minor cosmetic change, but seems like they should disappear if resizing is disabled.

[eduardo]

If you want hide makes more sense destroy the resizable or do it using theme: .ui-resizable-disabled .ui-resizable-handle { display: none; }

[machineghost]

Wait, so to disable and then re-enable the resizing of an element, I have to completely destroy and then recreate? That makes no sense; why pay the expense of destroying/recreating when we could keep everything and just not trigger the autoHide if disabled? I mean really, why even have "disabled" if you're going to tell people they have to destroy stuff?

Furthermore, the proposed fix/workaround of:

.ui-resizable-disabled .ui-resizable-handle { display: none; }

doesn't work at all, because the autoHide handler explicitly does "self._handles.show();" which overrides any style you could possibly add.

Could the devs please re-open this ticket? The *real* fix (IMESHO) is to simply have the autoHide hover function check whether the resizable is disabled, and if not return. This is super-easy to do, takes 2 lines, and solves the bug in a way that makes sense, not in a "suck it user, you're not doing what we're doing so you don't matter" kind of way ;-)

In other words this chunk of code:

.hover(function() {
	$(this).removeClass("ui-resizable-autohide");
	self._handles.show();
},
function(){
	if (!self.resizing) {
		$(this).addClass("ui-resizable-autohide");
		self._handles.hide();
	}
});

which comes a few lines after "if (o.autoHide)", should really be:

.hover(function() {
	if ($(this).parent.hasClass("ui-resizable-disabled")) return;
	$(this).removeClass("ui-resizable-autohide");
	self._handles.show();
},
function(){
	if ($(this).parent.hasClass("ui-resizable-disabled")) return;
	if (!self.resizing) {
		$(this).addClass("ui-resizable-autohide");
		self._handles.hide();
	}
});

I have tested this fix and it does solve the problem; please consider implementing it so that I (and everyone else who comes across this bug) don't have to run a "hacked" copy of the jQuery UI code, which is a pain to upgrade (also an uncountable number of users who will never see this bug report but will see this bug will be appreciate it).

[scottgonzalez]

machineghost, please file a new ticket since your (valid) complaint is specifically about the autoHide option.

[machineghost]

oh, sure thing; also, for posterity, my fix was dumb and had ".parent" when it shouldn't.