-
Notifications
You must be signed in to change notification settings - Fork 165
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
Applied some animation corrections for buildings when player is on low powered. #728
base: master
Are you sure you want to change the base?
Applied some animation corrections for buildings when player is on low powered. #728
Conversation
1. Allied Construction Yard(gacnst) 2. Allied Airforce Command Headquarters (gaairc) 3. Allied Ore Purifier (gaorep) Changed constructions yard and airforce command to not pause the animation when it is on low power (This is how it works on the original game) Changed the ore purifier to halt on animation when it is on low power (This is how it works on the original game)
Tbh, not sure if pausing the Airforce HQ isn't a bugfix over the original; will have to think about that. (Doesn't it look a bit weird when that doesn't pause but the Spy Sat does?) |
From what I originally understood, if the animation pauses, it is a visual indicator that you can't use it. However, Westwood is inconsistent on this rule because Ore purifier for example, has its animation paused but the passive effect still remains in effect regardless. For the airforce command, I think they didn't let it pause because you can still use the aircrafts? But then again this could also be one of the inconsistencies too. |
|
So what do people think about slowing down animations on structures whose effects get slowed down on low power to reflect it? (like production structures) |
Why slow down instead of stopping? |
To better reflect what happens to the underlying effect - some structures stop working (radar) and some slow down (production). So we can stop the animation on some and slow down the animation on others to make that clear. |
Ok that makes sense to me. (As long as it does look good ingame, 👍 ) |
Hi! Sounds good to me as well, I'll try to apply the changes tonight or tomorrow. |
…tion named idle-fans-low-power which is just like idle-fans but with larger ticks to make the animation appear slower
…ction yard's fan animation slow down when low power and reverted animation pauses on Airforce command and Ore Purifier
@penev92 and @abcdefg30 i reverted changes 2 and 3 to what it originally was and as for 1, I made some changes base on the discussion where if player is in LOWPOWER state, then it will slow down the fans. |
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.
I'll need to test, but the way it is set up looks correct.
On a side note, would you be willing to do this for other production structures that have idle (or production) animations?
Added safeguard lowpower condition in the Sequence idle-fans-low-power Co-authored-by: Pavel Penev <[email protected]>
Sure, I just need to know which structures need this implementation. |
Actually taking a quick look I can't spot any other candidates. |
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.
Looks good to me 👍
Thank you!
Modified the following rules:
Changed constructions yard and airforce command to not pause the animation when it is on low power (This is how it works on the original game)
Changed the ore purifier to halt on animation when it is on low power (This is how it works on the original game)
This pull request is related to the issue filed in #727