show current buff value on Devastation tooltip #46
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
adavidw
added a commit
to adavidw/FrozenCookies
that referenced
this pull request
Oct 23, 2020
* autoGodzamok, autoSL & autoRigidel (Mtarnuhal#45) * Fix Mtarnuhal#18 autoSL + autoRigidel Seems Auto harvest, and auto rigidel+harvest attempt to harvest a sugar lump just a teensy bit before it's actually ripe, therefore botching the harvest in half the cases. Game.timetoRipe returns a float number in some cases, so adding Math.ceil to the Game.timeToRipe in the equations should give it just enough leeway to make sure it computes the correct millisecond. (It's my theory anyway. The math seemed to have come out ok when I paper tested). With thnx to @Darkroman * autoGodzamok Sane & Insane option restored Attempt to restore the autoGodzamok Sane and Insane options. Has to be tested to make sure everything behaves as it should. * Update frozen_cookies.js * Attempt fix autoGodzamok Sane and Insane Second try And some minor changes to make it more efficient. * safeBuy safeBuy instead of visually change the buy-sell option. Changing the variable should be enough. * Attempt fix autoGodzamok Sane and Insane Third try Preventing the sell of cursors and farms while Devastation is active. * Finalizing autoGodzamok finalizing autoGodzamok * Update Version 1.8.1 * Simplifying autoGodzamok simplified autoGodzamok. - If AutoGodz is on and a click buff is active, but Devastation is NOT active: - sell the cursors and farms - if autoBuy is turned on, the user has already signaled their intent to have things purchased for them, so rebuy what you sold (up to the limit if any limit is set). - Otherwise, don't rebuy anything. * Remove unnecessary autoGodzamok preference Removed Sane and Insane mode from autoGodzamok cause they were removed. * autoGodzamok Reversed variables Reversed the variables of buy back * autoGodzamok wrong parameter Used the wrong parameter * autoGodzamok extra check Extra check if the limit doesn't exceed the current buildings. * autoGodzamok Forgot to add -1 to the buyback of farms * show current buff value on Devastation tooltip (Mtarnuhal#46) Co-authored-by: Mtarnuhal <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I went ahead and fixed the tooltip for the graphic for Godzamok's buff. Now when you hover over the graphic, it shows the current value of the buff.