Opened 8 years ago

Closed 8 years ago

Last modified 7 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:

Change History (3)

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

Priority: minorblocker
Status: newopen
Summary: Merge offset option into my and at optionsPosition: Merge offset option into my and at options

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

Resolution: fixed
Status: openclosed

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 7 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.