Skip to content
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

fix: add colored hexagon outlines on unit hover [#2242] #2497

Merged
merged 2 commits into from
Oct 2, 2023

Conversation

cesardlinx
Copy link
Contributor

I added color coded outline hexagons to be displayed on top of the black ones that are filled when hovering an inactive unit, as you can see in the gif below

ancient_beast5

Closes #2242

My wallet address is 0xb2717FC8dFcc42A75E06A38aa273C0b9F4Cba848

Hovering an inactive unit now should causes color coded outline hexagons
to be displayed on top of the black ones that are filled, to better
indicate unit size and hexagons occupied, as well as the player it belongs to

Closes FreezingMoon#2242
@vercel
Copy link

vercel bot commented Oct 1, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ancientbeast ✅ Ready (Inspect) Visit Preview Oct 2, 2023 4:25am

@ghost
Copy link

ghost commented Oct 1, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@DreadKnight
Copy link
Member

@cesardlinx Pretty good overall! Just a small regression: when targeting a lot of abilities (that usually involve area somehow), there's no more colored fill change for the affected hexagons.

See for example Abolished's Fiery Claw ability.

@DreadKnight DreadKnight marked this pull request as draft October 1, 2023 23:45
@cesardlinx
Copy link
Contributor Author

@DreadKnight Thank you! Good catch! I didn't notcied that, I think I fixed the issue, now that functionality is working as before as you can see it in the gif

ancient_beast_area

Feel free to reach out if there's anything else!

@DreadKnight DreadKnight marked this pull request as ready for review October 2, 2023 04:40
@DreadKnight
Copy link
Member

@cesardlinx That does the job, well done! Bounty sent 🪙

@DreadKnight DreadKnight merged commit e1507c1 into FreezingMoon:master Oct 2, 2023
1 check passed
@cesardlinx
Copy link
Contributor Author

Cool! Thank you 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

colored hexagon outlines on unit hover [bounty: 5 XTR]
2 participants