Skip to content

Commit 4e67172

Browse files
committed
Closes issue CesiumGS#5691
Some fixes to previous commit as @hpinkos recommended Replaced eventListener moveStart / moveEnd with Camera.changed event listener. Replaced setInterval with setTimeout function in that event.
1 parent 7248daf commit 4e67172

File tree

1 file changed

+4
-10
lines changed

1 file changed

+4
-10
lines changed

Apps/CesiumViewer/CesiumViewer.js

+4-10
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ define([
166166
}
167167

168168
var camera = viewer.camera;
169-
function saveCamera(camera) {
169+
function saveCamera() {
170170
var position = camera.positionCartographic;
171171
var hpr = '';
172172
if (defined(camera.heading)) {
@@ -178,17 +178,11 @@ define([
178178

179179
var updateTimer;
180180
if (endUserOptions.saveCamera !== 'false') {
181-
camera.moveStart.addEventListener(function() {
182-
if (!defined(updateTimer)) {
183-
updateTimer = window.setInterval(saveCamera, 1000, camera);
184-
}
185-
});
186-
camera.moveEnd.addEventListener(function() {
181+
camera.changed.addEventListener(function() {
187182
if (defined(updateTimer)) {
188-
window.clearInterval(updateTimer);
189-
updateTimer = undefined;
183+
window.clearTimeout(updateTimer);
190184
}
191-
saveCamera(camera);
185+
updateTimer = window.setTimeout(saveCamera, 1000);
192186
});
193187
}
194188

0 commit comments

Comments
 (0)