Merge pull request #838 from rasben/feature/fix-menu-positioning

mlh758 · web-flow · commit 6bbf1a6ce2ed · 2019-04-01T11:23:42.000-05:00
ISSUE 837: Use jquery offset instead of .position().
diff --git a/src/jquery.multiselect.js b/src/jquery.multiselect.js
@@ -1507,8 +1507,11 @@
         this.$menu.position(this.options.position);
       }
       else {
-        var pos = $button.position();
-        pos.top += this._savedButtonHeight;
+        var pos = {};
+
+        pos.top = $button.offset().top + this._savedButtonHeight;
+        pos.left = $button.offset().left;
+
         this.$menu.offset(pos);
       }
     },

Original file line number	Diff line number	Diff line change
`@@ -1507,8 +1507,11 @@`
`1507`	`1507`	`this.$menu.position(this.options.position);`
`1508`	`1508`	`}`
`1509`	`1509`	`else {`
`1510`		`- var pos = $button.position();`
`1511`		`- pos.top += this._savedButtonHeight;`
	`1510`	`+ var pos = {};`
	`1511`	`+`
	`1512`	`+ pos.top = $button.offset().top + this._savedButtonHeight;`
	`1513`	`+ pos.left = $button.offset().left;`
	`1514`	`+`
`1512`	`1515`	`this.$menu.offset(pos);`
`1513`	`1516`	`}`
`1514`	`1517`	`},`