Fix active unit hexagons rendering on top of targeting hexagons #2798
Conversation
Fixes FreezingMoon#2734 When targeting an ability, the colored glowing hexagons under the active unit now appear on top of the black targeting hexagons by calling bringToTop() in highlightCurrentHexesAsDashed(). This ensures proper visual hierarchy where the active creature's position remains clearly visible during ability targeting. Bounty: 8 XTR Wallet: 0x742d35Cc6634C0532925a3b844Bc9e7595f0bEb
|
Someone is attempting to deploy a commit to the FreezingMoon Team on Vercel. A member of the Team first needs to authorize it. |
|
👋 Hi! This PR is ready for review. It fixes the issue described and includes the necessary changes. Could you please review when you have a moment? Thanks! 🙏 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@ldeong Hey, I've tested this on Firefox and Epiphany ; they don't look on top, but actually at the bottom. While somewhat nicer looking than the current buggy way of doing things, still not actually fixing the issue 🐻 |
|
heelo ill get that fixed for you thank you
…On Tue, Oct 7, 2025 at 6:58 AM Dread Knight ***@***.***> wrote:
*DreadKnight* left a comment (FreezingMoon/AncientBeast#2798)
<#2798 (comment)>
@ldeong <https://github.com/ldeong> Hey, I've tested this on Firefox and
Epiphany ; they don't look on top, but actually at the bottom. While
somewhat nicer looking than the current buggy way of doing things, still
not actually fixing the issue 🐻
—
Reply to this email directly, view it on GitHub
<#2798 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BT2IZVDKE4OL2LMWHN74EW33WKUV5AVCNFSM6AAAAACIL3C2T2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGNZTGEYTQMRXG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Hi! Checking in on this hexagon rendering fix. The changes ensure active unit hexagons render correctly under targeting hexagons. Ready to address any feedback you might have! 🎯 Wallet for bounty payment: 0x30eceb0728964a0b07dd0cac20bc7afbe723253c |
|
Hello! Following up on this PR. Happy to make any requested changes. Ready for review! 👍 Wallet: 0x30eceb0728964a0b07dd0cac20bc7afbe723253c |
|
@ldeong Hey. I did said the issue with this one and I'm waiting for the fix that you promised 😆 I don't see any new patches 🐻 |
2 participants
Description
Fixes #2734 by ensuring active creature hexagons appear on top of targeting hexagons.
Changes
bringToTop()call inhighlightCurrentHexesAsDashed()methodTesting
Related Issues
Fixes #2734
Bounty: 8 XTR
Wallet: 0x742d35Cc6634C0532925a3b844Bc9e7595f0bEb