Opened 10 years ago
Closed 10 years ago
#8757 closed bug (fixed)
Draggable: Resizable handle with inner element does not work, when its also draggable
Reported by: | MatteS75 | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 1.10.0 |
Component: | ui.draggable | Version: | 1.9.1 |
Keywords: | haspatch | Cc: | |
Blocked by: | Blocking: |
Description
Creating a handle with an inner element renders the resize handle useless.
- Create a resizable div with an inner div with the handle classes setup
- Add another div inside the handle div
- Initialize the draggable
- Initialize the resizable
Now the resizable handle does not work. Removing the inner div, and it does work.
This might seems to be the same as http://bugs.jqueryui.com/ticket/8756, but it might not be. Fixing that bug might make this scenario fail still. Applying the workaround for this, and it still does not work: http://jsfiddle.net/MatteS75/A7N6L/
Change History (8)
comment:1 follow-up: 5 Changed 10 years ago by
Resolution: | → notabug |
---|---|
Status: | new → closed |
comment:2 Changed 10 years ago by
If its helping someone else, this is the code change that fixes it:
@@ -5909,7 +5909,7 @@ var o = this.options; // among others, prevent a drag on a resizable-handle - if (this.helper || o.disabled || $(event.target).is('.ui-resizable-handle')) + if (this.helper || o.disabled || $(event.target).is('.ui-resizable-handle') || $(event.target).closest('.ui-resizable-handle').length > 0) return false;
comment:3 Changed 10 years ago by
Resolution: | notabug |
---|---|
Status: | closed → reopened |
Reopening based on pending PR.
comment:4 Changed 10 years ago by
Component: | ui.resizable → ui.draggable |
---|---|
Status: | reopened → open |
comment:5 Changed 10 years ago by
Replying to scott.gonzalez:
I'm going to close this as not a bug since this isn't something we support. If this is still broken when (if) the other ticket is resolved, then feel free to file a new ticket.
Why was this reopened? Not sure what the "pending" PR is?
comment:6 Changed 10 years ago by
Keywords: | haspatch added |
---|
I'm waiting on a new PR based on https://github.com/jquery/jquery-ui/pull/807#issuecomment-10081536
If that doesn't come soon, we can just update his commit and land it.
comment:7 Changed 10 years ago by
Summary: | Resizable handle with inner element does not work, when its also draggable → Draggable: Resizable handle with inner element does not work, when its also draggable |
---|
comment:8 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Draggable: made handles work with complex markup. Fixes #8757 - Draggable: Resizable handle with inner element does not work, when its also draggable
Changeset: cf9fbce13931732dc4b06d491ef2a01c4921faf1
I'm going to close this as not a bug since this isn't something we support. If this is still broken when (if) the other ticket is resolved, then feel free to file a new ticket.