Skip to content
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

Fix night shading in 2D #6902

Merged
merged 3 commits into from
Aug 8, 2018
Merged

Fix night shading in 2D #6902

merged 3 commits into from
Aug 8, 2018

Conversation

bagnell
Copy link
Contributor

@bagnell bagnell commented Aug 8, 2018

Fixes #4122.

@cesium-concierge
Copy link

Thanks for the pull request @bagnell!

  • ✔️ Signed CLA found.

Reviewers, don't forget to make sure that:

  • Cesium Viewer works.
  • Works in 2D/CV.
  • Works (or fails gracefully) in IE11.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.

🌍 🌎 🌏

@@ -14,6 +14,7 @@ Change Log
* The Viewer widget now takes terrain altitude into account when zooming or flying to imagery layers.
* Fixed bug that caused a new `ClippingPlaneCollection` to be created every frame when used with a model entity [#6872](https://github.com/AnalyticalGraphicsInc/cesium/pull/6872)
* Fixed crash when rendering translucent objects when all shadow maps in the scene set `fromLightSource` to false. [#6883](https://github.com/AnalyticalGraphicsInc/cesium/pull/6883)
* Fixed night shading in 2D. [#4122](https://github.com/AnalyticalGraphicsInc/cesium/issues/4122)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And Columbus View?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nevermind, didn't realize it already worked there.

@hpinkos
Copy link
Contributor

hpinkos commented Aug 8, 2018

Thanks @bagnell!

@hpinkos hpinkos merged commit 9eef895 into master Aug 8, 2018
@hpinkos hpinkos deleted the lighting-2d branch August 8, 2018 21:47
# 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.

4 participants