Skip to content

Closes issue #5691 #5699

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 4 commits into from
Closed

Closes issue #5691 #5699

wants to merge 4 commits into from

Conversation

avalan4e57
Copy link

No description provided.

The issue is about fixing safari crash on mouse move because of using
replaceState to often.

There was a syntax error in using setTimeout function where interval
value was on the third place in func parameters list when it should
be on second.
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.
Set 'updateTimer = undefined' after 'clearTimeout(updateTimer)' call
@hpinkos
Copy link
Contributor

hpinkos commented Aug 1, 2017

@avalan4e57 can you please send us a Contributor's License Agreement so we can review and merge this? See https://github.com/AnalyticalGraphicsInc/cesium/blob/master/CONTRIBUTING.md#contributor-license-agreement-cla

@avalan4e57
Copy link
Author

@hpinkos done!

@avalan4e57
Copy link
Author

@hpinkos I’ve thought a little about it. And came to conclusion that I don’t want to contribute to AGI projects. Either I get paid for my code or I have my rights on it. Thank you for understanding and sorry for waisting your time.

@pjcozzi
Copy link
Contributor

pjcozzi commented Aug 1, 2017

No worries @avalan4e57! Thanks for considering to support the project, we hope you are having great success with your development.

@pjcozzi pjcozzi closed this Aug 1, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants