cartodb/Leaflet
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Performance: don’t set width/height on SVG if unchanged. Saves a costly layout/repaint in some situations.

Browse Source
This commit is contained in:
Fabian Zeindl 2014-06-17 15:01:42 +02:00
parent e522582da3
commit 132e5b6f7c
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/layer/vector/SVG.js
View File

@ -31,11 +31,17 @@ L.SVG = L.Renderer.extend({
L.DomUtil.setPosition(container, b.min);
// update container viewBox so that we don't have to change coordinates of individual layers
container.setAttribute('width', size.x);
container.setAttribute('height', size.y);
container.setAttribute('viewBox', [b.min.x, b.min.y, size.x, size.y].join(' '));
// set size of svg-container if changed
if (!this._svgSize || !this._svgSize.equals(size)) {
this._svgSize = size;
container.setAttribute('width', size.x);
container.setAttribute('height', size.y);
}
// movement: update container viewBox so that we don't have to change coordinates of individual layers
L.DomUtil.setPosition(container, b.min);
container.setAttribute('viewBox', [b.min.x, b.min.y, size.x, size.y].join(' '));
if (L.Browser.mobileWebkit) {
pane.appendChild(container);
}