Skip to main content

Bug Tracker

Side navigation

#2525 closed enhancement (invalid)

Opened March 15, 2008 05:46PM UTC

Closed July 11, 2010 06:39PM UTC

Standardised way to pass coordinates to plugins.

Reported by: xover Owned by: rworth
Priority: minor Milestone:
Component: ui Version:
Keywords: Cc:
Blocked by: Blocking:

Looking at Draggable's ''cursorAt'' and Dialog's ''position'' options I note that they use different ways to pass a set of coordinates to the function. Dialog uses an array of offsets from top left, where Draggable uses a hash of either {top: n, left: m} or {bottom: n, right: m} (etc.).

There should probably be a common way to pass coordinates to plugins; possibly by making Draggable's way the default with Dialog's way as a convenience shortcut. Dialog's method can also be extended to allow negative offsets to be relative to the bottom and right edges.

Attachments (0)
Change History (5)

Changed March 15, 2008 05:52PM UTC by rworth comment:1

I recommend that both could accept either an array [x, y] or a hash {top, bottom, left, right}

Changed March 17, 2008 09:34AM UTC by paul comment:2

owner: paulrworth

Draggables is updated to also take an array now Could you please continue with dialog?

Changed May 04, 2008 08:55AM UTC by paul comment:3

priority: majorminor

To make it easier to update the values later on internal API wise, I suggest to use hashes only.

Changed May 05, 2008 08:56AM UTC by paul comment:4

type: bugenhancement

Changed July 11, 2010 06:39PM UTC by dmethvin comment:5

resolution: → invalid
status: newclosed

This should be filed in the UI bug tracker if still relevant.