Skip to content

Commit

Permalink
revert iOS fix (it wasn't effective)
Browse files Browse the repository at this point in the history
still looking for a working fix
  • Loading branch information
FezVrasta committed Apr 12, 2016
1 parent 90fa8d2 commit 592f4c7
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/popper.js
Original file line number Diff line number Diff line change
Expand Up @@ -418,8 +418,8 @@
// NOTE: 1 DOM access here
_updateBound = this.update.bind(this);
root.addEventListener('resize', _updateBound);
// if the boundariesElement is window or the popper position is fixed, we don't need to listen for the scroll event
if (this._options.boundariesElement !== 'window' && this.state.position !== 'fixed') {
// if the boundariesElement is window we don't need to listen for the scroll event
if (this._options.boundariesElement !== 'window') {
var target = getScrollParent(this._trigger);
// here it could be both `body` or `documentElement` thanks to Firefox, we then check both
if (target === root.document.body || target === root.document.documentElement) {
Expand All @@ -438,7 +438,7 @@
Popper.prototype._removeEventListeners = function() {
// NOTE: 1 DOM access here
root.removeEventListener('resize', _updateBound);
if (this._options.boundariesElement !== 'window' && this.state.position !== 'fixed') {
if (this._options.boundariesElement !== 'window') {
var target = getScrollParent(this._trigger);
// here it could be both `body` or `documentElement` thanks to Firefox, we then check both
if (target === root.document.body || target === root.document.documentElement) {
Expand Down

0 comments on commit 592f4c7

Please sign in to comment.