Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix Billboard.computeScreenSpacePosition #3976

Merged
merged 3 commits into from
May 30, 2016

Conversation

bagnell
Copy link
Contributor

@bagnell bagnell commented May 27, 2016

Fixes #3920.

Swap screen space position y coordinate so it increases from top to bottom. Also fix the calculation in 2D for labels.

…ottom. Also fix the calculation in 2D for labels.
@pjcozzi
Copy link
Contributor

pjcozzi commented May 28, 2016

@e-andersson can you please test this? We would like to include this in Cesium 1.22 in a few days on June 1? The branch is hosted here: http://cesium-dev.s3-website-us-east-1.amazonaws.com/cesium/billboard-screen-space-position/

@pjcozzi
Copy link
Contributor

pjcozzi commented May 28, 2016

@bagnell update CHANGES.md

@e-andersson
Copy link
Contributor

e-andersson commented May 30, 2016

@pjcozzi Did some quick testing (Sandcastle + our application) and it appears to work with the changes.

@pjcozzi
Copy link
Contributor

pjcozzi commented May 30, 2016

Thanks for testing @e-andersson!

@pjcozzi pjcozzi merged commit 269d14e into master May 30, 2016
@pjcozzi pjcozzi deleted the billboard-screen-space-position branch May 30, 2016 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Billboard.prototype.computeScreenSpacePosition returning wrong y position
3 participants