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

Added simple shadows sandcastle #3876

Merged
merged 4 commits into from
May 19, 2016
Merged

Added simple shadows sandcastle #3876

merged 4 commits into from
May 19, 2016

Conversation

lilleyse
Copy link
Contributor

For #2594

The simple sandcastle demo for shadows. There are a few places where I access scene directly, which I shouldn't do for a Showcase demo. I can maybe make these accessible through Viewer instead.

var scene = viewer.scene;
var shadowMap = scene.shadowMap;
shadowMap._maxmimumDistance = 10000.0;
scene.globe.castShadows = true;
Copy link
Contributor

Choose a reason for hiding this comment

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

globe is also available off of viewer.

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 23, 2016

Take a screenshot for this for the Sandcastle gallery.

@pjcozzi
Copy link
Contributor

pjcozzi commented Apr 23, 2016

  • Add a toggle to turn shadows on/off so users can clearly see them. We'll also want a toggle for soft shadows and any other common shadows setting. The idea is that users can come to this sample to learn the shadow properties they need to set.
  • Add an option for terrain and primitives to cast/receive shadows
  • Change the default view/model/time so that shadows are blatantly obvious. At least at this time, the shadow on the aircraft is not obvious.

@lilleyse
Copy link
Contributor Author

Updated.

@pjcozzi
Copy link
Contributor

pjcozzi commented May 18, 2016

When I switch the view from Exton to Half Dome, the aircraft stops animating. Is that intentional?

@lilleyse
Copy link
Contributor Author

When I switch the view from Exton to Half Dome, the aircraft stops animating. Is that intentional?

No... I'll look into that.

@pjcozzi
Copy link
Contributor

pjcozzi commented May 18, 2016

Otherwise, this is great.

Let me know when to merge.

@mramato
Copy link
Contributor

mramato commented May 18, 2016

Even with a maximumed browser window, I'm getting run in with the standard toolbar:

image

@lilleyse
Copy link
Contributor Author

This should be ready now. I vertically aligned the buttons.

@pjcozzi pjcozzi merged commit 0d39ddc into shadows May 19, 2016
@pjcozzi pjcozzi deleted the shadows-simple-sandcastle branch May 19, 2016 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants