Fix setTokenOpacity to update token for players. #108
Merged
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
* Added Submit a Question template with support links. * Removed Mobile related sections * Added MapTool version section for Bug Report
* Create CODE_OF_CONDUCT.md * Update build.gradle removed markdown files from spotless
…d jacoco to build.gradle (#104)
kayila
approved these changes
Oct 12, 2018
JamzTheMan
added
bug
MapTool
|
This is now live with 1.4.5.4! Thanks for the PR! |
JamzTheMan
added a commit
that referenced
this pull request
Nov 14, 2018
* Fixed #4 (#95) Created next-release branch for testing. * Updating next-release (#105) * Update issue templates * Update issue templates * Added Submit a Question template with support links. * Removed Mobile related sections * Added MapTool version section for Bug Report * Create CODE_OF_CONDUCT.md (#97) * Create CODE_OF_CONDUCT.md * Update build.gradle removed markdown files from spotless * Added some files to .gitignore, removed auto-generated file, and added jacoco to build.gradle (#104) * Fix setTokenOpacity to update token for players. (#108) * Update issue templates * Update issue templates * Added Submit a Question template with support links. * Removed Mobile related sections * Added MapTool version section for Bug Report * Create CODE_OF_CONDUCT.md (#97) * Create CODE_OF_CONDUCT.md * Update build.gradle removed markdown files from spotless * Added some files to .gitignore, removed auto-generated file, and added jacoco to build.gradle (#104) * Fix setTokenOpacity to update token for players. * Ran spotlessCheck to fix style issues. * Bug fix release 1.4.5.4 (#114) * Update issue templates * Update issue templates * Added Submit a Question template with support links. * Removed Mobile related sections * Added MapTool version section for Bug Report * Create CODE_OF_CONDUCT.md (#97) * Create CODE_OF_CONDUCT.md * Update build.gradle removed markdown files from spotless * Added some files to .gitignore, removed auto-generated file, and added jacoco to build.gradle (#104) * Enhancement - getMoveCount function * Added optional 2nd parameter to getMoveCount(returnFractional, userTerrainModifiers) macro function to return distanced traveled using terrain modifiers. First parameter, returnFractional, if true, ignores 2nd parameter and returns 1 if last move was a fraction, eg for 1-2-1 movement tracking. * 113: Macro function getMoveCount() does not return terrain modifier costs * Hack to get JDK 10 from my server and manually extract for AppVeyor - Linux. Hopefully this works! Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * 113: Macro function getMoveCount() does not return terrain modifier costs * Hack to get JDK 10 from my server and manually extract for AppVeyor - Linux. Hopefully this works this time! Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * 113: Macro function getMoveCount() does not return terrain modifier costs * Hack to get JDK 10 from my server and manually extract for AppVeyor - Linux. Hopefully this works this time! * helps to untar java dir not under project dir Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * 113: Macro function getMoveCount() does not return terrain modifier costs * Debugging deploy on Linux Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * 113: Macro function getMoveCount() does not return terrain modifier costs * Debugging deploy on linux Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * 113: Macro function getMoveCount() does not return terrain modifier costs * Debugging deploy on linux Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * 113: Macro function getMoveCount() does not return terrain modifier costs * Debugging deploy on linux Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * #113: Macro function getMoveCount() does not return terrain modfier costs * Changelog updated, Ready for release 1.4.5.4! Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net> * Appveyor cleanup... * Changelog updates. * Removing fix for #95 Task-Url: #113 Signed-off-by: Jamz <Jamz@Nerps.net>
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.
Fix for issue #107. Instead of directly returning of
setTokenOpacity, the result is captured andputTokenfunctions are run before returning, similar to other functions which alter how a token is displayed.