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

Fixed map preview fade-in for weathers that set BLDALPHA. #23

Open
wants to merge 1 commit into
base: map-previews
Choose a base branch
from

Conversation

aarant
Copy link

@aarant aarant commented Sep 17, 2023

Fixes Shade weather being applied to the map preview for forest type transitions.
Fixes map preview interaction with weathers that initialize BLDALPHA.

Description

If a weather sets blend parameters in its InitVars or InitAll functions, these interfere with the map preview's fade-in. Also, because the map preview grabs the state of BLDALPHA before this happens, it doesn't properly restore the blend settings for the weather afterwards.

This is only an issue for people using my lighting code or who have made other similar changes to weathers. However I can't really fix it on my end. So I just added a different way to reset the blend params after the fade-in, and made the task set them to 16, 0 until the fade-in starts, to fix these issues.

Discord contact info

merrp#0001

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