ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Take view padding into account for calculateExtent()

Browse Source
This commit is contained in:
Andreas Hocevar
2021-01-22 20:08:05 +01:00
parent 9e6ec55758
commit 900107df1d
2 changed files with 16 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/ol/View.js
View File

@@ -921,7 +921,7 @@ class View extends BaseObject {
* @return {import("./extent.js").Extent} Extent.
*/
calculateExtentInternal(opt_size) {
const size = opt_size || this.getViewportSize_();
const size = opt_size || this.getViewportSizeMinusPadding_();
const center = /** @type {!import("./coordinate.js").Coordinate} */ (this.getCenterInternal());
assert(center, 1); // The view center is not defined
const resolution = /** @type {!number} */ (this.getResolution());

15
test/spec/ol/view.test.js
View File

@@ -1622,6 +1622,21 @@ describe('ol.View', function () {
expect(extent[2]).to.roughlyEqual(51200, 1e-9);
expect(extent[3]).to.roughlyEqual(25600, 1e-9);
});
it('works with a view padding', function () {
const view = new View({
resolutions: [1],
zoom: 0,
center: [0, 0],
padding: [10, 20, 30, 40],
});
let extent = view.calculateExtent();
expect(extent).to.eql([-20, -30, 20, 30]);
view.padding = [0, 0, 0, 0];
view.setCenter([0, 0]);
extent = view.calculateExtent();
expect(extent).to.eql([-50, -50, 50, 50]);
});
});
describe('#getViewportSize_()', function () {