-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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 TS crane overlay's last frame hiding idle overlay #9377
Conversation
Are all the other overlay traits going to have similar problems? |
@@ -35,6 +35,7 @@ public class WithBuildingPlacedOverlay : INotifyBuildComplete, INotifySold, INot | |||
{ | |||
readonly Animation overlay; | |||
bool buildComplete; | |||
bool hide; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be a bit more consistent with other code to call this visible
and invert the checks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok.
Possibly. I'll take a look. |
5b9712c
to
e9c2c4b
Compare
This was an idle animation in the original, and this and "platform" cannot be displayed at the same time anyway. Also increased Tick rate to match idle light.
e9c2c4b
to
4aaa628
Compare
Fixed |
👍 As a nice side effect this also makes the cloaking look better. |
Confirmed fixes 👍. Thanks. |
Fixed TS crane overlay's last frame hiding idle overlay
TS crane overlay's last frame no longer overlaps idle overlay.
WithBuildingPlacedOverlay
is now only visible while playing the animation, rather than sticking around.Fixes #9319.