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

Don't append/remove the SVG container

Browse Source 
Tested this on IOS7, which has 85% adoption and I see no flicker.
Reduces layout and paint-times.
This commit is contained in:
Fabian Zeindl 2014-06-17 16:10:21 +02:00 committed by John Firebaugh
parent c6dea37d86
commit 4f06ac30b0
1 changed files with 1 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -21,13 +21,7 @@ L.SVG = L.Renderer.extend({
var b = this._bounds, var b = this._bounds,
size = b.getSize(), size = b.getSize(),
container = this._container, container = this._container;
pane = this.getPane();
// hack to make flicker on drag end on mobile webkit less irritating
if (L.Browser.mobileWebkit) {
pane.removeChild(container);
}
L.DomUtil.setPosition(container, b.min); L.DomUtil.setPosition(container, b.min);
@ -41,10 +35,6 @@ L.SVG = L.Renderer.extend({
// movement: update container viewBox so that we don't have to change coordinates of individual layers // movement: update container viewBox so that we don't have to change coordinates of individual layers
L.DomUtil.setPosition(container, b.min); L.DomUtil.setPosition(container, b.min);
container.setAttribute('viewBox', [b.min.x, b.min.y, size.x, size.y].join(' ')); container.setAttribute('viewBox', [b.min.x, b.min.y, size.x, size.y].join(' '));
if (L.Browser.mobileWebkit) {
pane.appendChild(container);
}
}, },
// methods below are called by vector layers implementations // methods below are called by vector layers implementations