Merge pull request #5644 from select2/GH-5584
Fixes jQuery migrate error when getting offset when dropdownParent not in document
This commit is contained in:
commit
7c4131bc27
9
src/js/select2/dropdown/attachBody.js
vendored
9
src/js/select2/dropdown/attachBody.js
vendored
@ -190,7 +190,14 @@ define([
|
|||||||
$offsetParent = $offsetParent.offsetParent();
|
$offsetParent = $offsetParent.offsetParent();
|
||||||
}
|
}
|
||||||
|
|
||||||
var parentOffset = $offsetParent.offset();
|
var parentOffset = {
|
||||||
|
top: 0,
|
||||||
|
left: 0
|
||||||
|
};
|
||||||
|
|
||||||
|
if ($.contains(document.body, $offsetParent[0])) {
|
||||||
|
parentOffset = $offsetParent.offset();
|
||||||
|
}
|
||||||
|
|
||||||
css.top -= parentOffset.top;
|
css.top -= parentOffset.top;
|
||||||
css.left -= parentOffset.left;
|
css.left -= parentOffset.left;
|
||||||
|
@ -175,3 +175,52 @@ test('dropdown is positioned down with absolute offsets', function (assert) {
|
|||||||
'There should not be an extra left offset'
|
'There should not be an extra left offset'
|
||||||
);
|
);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
test('dropdown is positioned even when not in document', function (assert) {
|
||||||
|
var $ = require('jquery');
|
||||||
|
var $select = $('<select></select>');
|
||||||
|
|
||||||
|
var $container = $('<span>test</span>');
|
||||||
|
var container = new MockContainer();
|
||||||
|
|
||||||
|
var Utils = require('select2/utils');
|
||||||
|
var Options = require('select2/options');
|
||||||
|
|
||||||
|
var Dropdown = require('select2/dropdown');
|
||||||
|
var AttachBody = require('select2/dropdown/attachBody');
|
||||||
|
|
||||||
|
var DropdownAdapter = Utils.Decorate(Dropdown, AttachBody);
|
||||||
|
|
||||||
|
var dropdown = new DropdownAdapter($select, new Options({
|
||||||
|
dropdownParent: $('html')
|
||||||
|
}));
|
||||||
|
|
||||||
|
var $dropdown = dropdown.render();
|
||||||
|
|
||||||
|
assert.equal(
|
||||||
|
$dropdown[0].style.top,
|
||||||
|
0,
|
||||||
|
'The drodpown should not have any offset before it is displayed'
|
||||||
|
);
|
||||||
|
|
||||||
|
dropdown.bind(container, $container);
|
||||||
|
dropdown.position($dropdown, $container);
|
||||||
|
dropdown._showDropdown();
|
||||||
|
|
||||||
|
assert.ok(
|
||||||
|
dropdown.$dropdown.hasClass('select2-dropdown--below'),
|
||||||
|
'The dropdown should be forced down'
|
||||||
|
);
|
||||||
|
|
||||||
|
assert.equal(
|
||||||
|
$dropdown.css('top'),
|
||||||
|
'0px',
|
||||||
|
'The offset should be 0px at the top'
|
||||||
|
);
|
||||||
|
|
||||||
|
assert.equal(
|
||||||
|
$dropdown.css('left'),
|
||||||
|
'0px',
|
||||||
|
'The offset should be 0px on the left'
|
||||||
|
);
|
||||||
|
});
|
||||||
|
Loading…
Reference in New Issue
Block a user