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

Dynamic lights on objects #12

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

floral-qua-floral
Copy link

I added a section to the readme about how to use this, but the gist of it is there's now a ds_list named dynamic_lights in the stage object. The dynamic lighting object uses that list to render lights the same way it uses article12s. I also made it so that Zetterburn, Forsburn, and Sein all have a faint glow when they're in the dark. Zetterburn's glow is pretty straightforward. It shows up when he's in an area that's relatively dark. Forsburn's glow is dimmer than his brother's, since his fiery mane isn't quite as big and magnificent. It's also yellowy orange as opposed to Zetter's more red glow. Sein's glow is blue and very large, but quite dim; although it shows up in even the slightest darkness. Additionally, Sein's glow is temporarily transferred to Ori during team-up attacks (since it turns out the Sein object doesn't follow Ori's position during those moves). Here are some screenshots of Zetter, Forsburn, and Ori & Sein:

image

I also noticed & fixed a small bug while in the process of making this change - it looks like in other_init.gml it checks if p_name is "Ori", but it should be checking for "Ori & Sein". Before I made this change it seems that Ori wasn't running the code to set their Character Attributes.

The reference to the dynamic lights list is stored in the player, not in the projectile. I forgot to include the "player_id" here, so this code would throw an error since it'd be checking in the projectile for a "dynamic_lights" variable.
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.

1 participant