Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#6981 closed enhancement (fixed)

Position: Merge offset option into my and at options

Reported by: adovenmuehle Owned by:
Priority: blocker Milestone: 1.9.0
Component: ui.position Version: 1.8.9
Keywords: Cc:
Blocked by: Blocking:

Description

See ​Position API Redesign Forum Post

Change History (3)

comment:1 Changed 12 years ago by Scott González

Priority: minor → blocker
Status: new → open
Summary: Merge offset option into my and at options → Position: Merge offset option into my and at options

comment:2 Changed 12 years ago by Scott González

Resolution: → fixed
Status: open → closed

Position: Merged offset option into my and at options and added support for percentage-based offsets. Fixes #6981 - Position: Merge offset option into my and at options. Fixes #7028 - Position: Allow percent-based offsets.

Changeset: 6f051d5d6a8eaf6fcc2a08ff68dbf5378029abc2

comment:3 Changed 10 years ago by olejorgenb

Gets kinda awkward when you have dynamic offsets.. 

function offsetString(value) {
  if(value >= 0)
    return "+"+value;
  else
    return ""+value; // -45 ==> "-45"
}
// ...
elem.position({
  my: "right"+offsetString(dynamicOffset) + " center",
// ...
});

Easily solvable by a utility function though

Note: See TracTickets for help on using tickets.