Ticket #5181 (closed bug: wontfix)

Opened 3 years ago

Last modified 3 years ago

No test for Constraints when Array passed : Patch included

Reported by: macarthy Owned by:
Priority: minor Milestone: 1.9.0
Component: ui.draggable Version: 1.8rc1
Keywords: Cc:
Blocking: Blocked by:

Description

The docs mention that an array can be used to constrain an item being dragged. I believe there is a bug, in constraining the element in that the Constraints do not take into consideration the height/width of the element being dragged.

I've created a patch with 2 new tests, one passes (when dragged up and left) and one fails (down and right).

My first bug report and patch, so be gentle :-)

From the docs:

Constrains dragging to within the bounds of the specified element or region. Possible string values: 'parent', 'document', 'window', [x1, y1, x2, y2].

Attachments

draggable_options.patch Download (1.4 KB) - added by macarthy 3 years ago.
new tests for contrain by array

Change History

Changed 3 years ago by macarthy

new tests for contrain by array

comment:1 Changed 3 years ago by scott.gonzalez

  • Milestone changed from TBD to 1.8

comment:2 Changed 3 years ago by scott.gonzalez

  • Status changed from new to closed
  • Resolution set to wontfix

Containing by an array will be dropped in the draggable rewrite. This should be built as an extension by modifying ui.position in the drag event.

Note: See TracTickets for help on using tickets.