Issue_1250 #1290
Merged
Issue_1250 #1290
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
…hen filled hex when hovered. Also includes pointer fix for 1289
|
@ShaneWalsh Good job! Opened some more issues to improve overall consistency for ability targeting. |
CyberBishop
pushed a commit
to CyberBishop/AncientBeast
that referenced
this pull request
Apr 20, 2023
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.
colored hexagon outlines for targets #1250 : resolved by making the reachable creatures have a hallow hex when chosen as targets, and a filled hex when hovered.
Also resolved #1289 as part of the same change, by making the onHover check if the target tile is reachable. If it is its a pointer, because u can execute an action on them, else its the arrow cursor because the hex is not reachable.
So if you have a creature with a 1 range attack, and a creature is within one hex but the creature has multiple hex's, only the closest hex will have a hallow hex colored, and when you hover on it should have pointer cursor. If you hover on the other hex they should have the arrow cursor because they are not selectable.
let me know what you think.