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

Add Resource information to AmbientLight Docs #7540

Closed
wants to merge 2 commits into from

Conversation

midnadimple
Copy link

Obejctives

Fixes #7465.

Solution

The documentation now tells the reader that AmbientLight is a Resource that has a default color of white and a default brightness of 0.05

Addresses bevyengine#7465.

The documentation now tells the reader that AmbientLight is a Resource that has a default color of white and a default brightness of 0.05
@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2023

Welcome, new contributor!

Please make sure you've read our contributing guide and we look forward to reviewing your pull request shortly ✨

@github-actions
Copy link
Contributor

github-actions bot commented Feb 7, 2023

Example alien_cake_addict failed to run, please try running it locally and check the result.

@superdump superdump added C-Docs An addition or correction to our documentation A-Rendering Drawing game state to the screen labels Feb 7, 2023
crates/bevy_pbr/src/light.rs Outdated Show resolved Hide resolved
@SpecificProtagonist
Copy link
Contributor

Thanks. Looks like bevy_pbr isn't correct either. I'm not sure, might need to be crate::PbrPlugin, but I can't check right now.

@rparrett
Copy link
Contributor

Looks like similar work was merged in #10712 and at least one other PR improving AmbientLight docs and this has unfortunately been forgotten.

I'm going to close this up, but please feel free to reopen and fix it up or open a new PR.

@rparrett rparrett closed this Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rendering Drawing game state to the screen C-Docs An addition or correction to our documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

AmbientLight docs do not explain how to use the struct
5 participants