Search and Top Navigation
#14912 open feature ()
Opened January 30, 2016 09:29AM UTC
Last modified February 03, 2016 05:46PM UTC
Draggable: Allow setting containment during drag
Reported by: | Mrten | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | none |
Component: | ui.draggable | Version: | 1.11.4 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
If you set a containment in the start event instead of in the default options, like this:
https://jsfiddle.net/8wzdt8z5/8/
the intended containment between cell 2 and 4 only works the second time you drag cell 3.
If you add an (obvious internal) call to _setContainment it works the first time (see fiddle).
Attachments (0)
Change History (3)
Changed January 30, 2016 09:32AM UTC by comment:1
Changed February 02, 2016 02:18AM UTC by comment:2
summary: | Setting containment in start works only the second time → Draggable: Allow setting containment in the start event |
---|---|
type: | bug → feature |
It's not that this works the second time, it's that the call from the first time has been applied.
This option is not meant to be updated during a drag. I'm not sure if we want to add support for this either. We'll have to discuss this within the team.
In the meantime, you can accomplish this with a custom containment via the drag
event. See https://jsfiddle.net/8wzdt8z5/9/
Changed February 03, 2016 05:46PM UTC by comment:3
status: | new → open |
---|---|
summary: | Draggable: Allow setting containment in the start event → Draggable: Allow setting containment during drag |
We're tentatively accepting this feature for the rewrite. However, the proposed patch is too limited since it only supports one specific time that the option can be changed. A proper implementation would handle changes to containment
at any point.
An obvious fix for this is: