#6948 closed bug (fixed)
Resizable autoHide Doesn't Work With Resizable's disable
Reported by: | machineghost | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.8.13 |
Component: | ui.resizable | Version: | 1.8.9 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Ticket #3237 already has all the description of this issue, as well as code for a fix (in a comment I just added), but I don't have the power to re-open that issue, and I don't want this easily-fixable bug to get missed, so I'm filing this dupe. Hopefully someone with higher privileges can delete this bug as a dupe and instead re-open (or at the very least respond to) that one. Thanks!
Change History (7)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
Component: | ui.core → ui.resizable |
---|---|
Status: | new → open |
comment:3 Changed 13 years ago by
Scott suggested that I file a new bug for this, and since I already have, please do not close this as a dupe after all. However, please do refer to my comment on 3237 about both the issue and the fix. Also, please refer to my follow up comment which explains that I'm a moron who submitted a bad fix; the real fix shouldn't have .parent, so it should be a matter of changing ...
.hover(function() { $(this).removeClass("ui-resizable-autohide"); self._handles.show(); }, function(){ if (!self.resizing) { $(this).addClass("ui-resizable-autohide"); self._handles.hide(); } });
to:
.hover(function() { if ($(this).is(".ui-resizable-disabled")) return; $(this).removeClass("ui-resizable-autohide"); self._handles.show(); }, function(){ if ($(this).is(".ui-resizable-disabled")) return; if (!self.resizing) { $(this).addClass("ui-resizable-autohide"); self._handles.hide(); } });
comment:5 Changed 12 years ago by
Created a demo of the issue here: http://jsfiddle.net/A46AY/
Commit to fix is here: https://github.com/carsonmcdonald/jquery-ui/commit/0e2bc5efe217681c006ae7a16be21520e6563da6
comment:6 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Resizable: Skip autohide hover action when resizable is disabled. Fix #6948 - Resizable autoHide Doesn't Work With Resizable's disable (cherry picked from commit 0e2bc5efe217681c006ae7a16be21520e6563da6)
Changeset: 38af07ec4e916bc5fe1268b9878a21801368a78d
comment:7 Changed 12 years ago by
Milestone: | 1.9 → 1.8.13 |
---|
Doh, and I filed it under the wrong component, sorry. Unfortunately I can't fix it because (even though I created this ticket) I don't have permissions to modify it. Sorry for the inconvenience; y'all really might want to consider slightly loosening edit permissions (at least on stuff people just filed two seconds ago) :-)