Skip to content

fixed flickering of priest 2 (player 2) when escorting with scavenger #2729

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yousefsassy
Copy link
Contributor

this fixes issue : #2698

issue:

When using the Scavenger’s Escort ability on both Player 1’s (Red) and Player 2’s (Blue) Priests, player 1(red) was escorted successfully but when we try to escort the second player (priest blue) via the same scavenger it filickers between grey and blue which is not the correct behavior it should stay blue.
here is before :
436785323-93f121a7-ee0f-47cd-ab00-7c1ff56a93c6

and here is after :

Screen.Recording.2025-05-13.at.10.02.55.PM.mov

Root Cause:

The flickering was caused by G.grid.previewCreature(trgPos, targetData, trg.player, true) being called after applying the manual displayVisualState() and overlayVisualState(). The previewCreature method was continuously overriding the target’s hex visuals with a default semi-transparent or neutral (grey) model.

Fix:

We commented out the call to:

G.grid.previewCreature(trgPos, targetData, trg.player, true);

in the select() function of the Scavenger’s Escort logic. This prevents the preview function from interfering with the manually applied team-colored visual states (playerX), effectively eliminating the flicker.

Copy link

vercel bot commented May 13, 2025

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

Name Status Preview Updated (UTC)
ancientbeast ✅ Ready (Inspect) Visit Preview May 13, 2025 9:36pm

@DreadKnight
Copy link
Member

@yousefsassy I've tested it and I find it rather hacky, as it's one step forward, one step back in the sense the PR should actually fix the Dark Priest preview during Escort Service ability, not have that disabled...
Will mark as draft for now and wait for a proper fix 🐻

@DreadKnight DreadKnight marked this pull request as draft May 14, 2025 11:24
@DreadKnight
Copy link
Member

@yousefsassy Hey! Any progress with this one? 🐻

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.

2 participants