Skip to content

showcase ability range on click #2695

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 2 commits into
base: master
Choose a base branch
from

Conversation

Kevin4335
Copy link

@Kevin4335 Kevin4335 commented Apr 23, 2025

Shows a preview for non-melee abilities that disappears after a delay when clicking an ability when it is unavailable. Changes the way handling abilities work with a new isPreview condition for query. Possible phase out the strict way of disabling abilities after 1 use and simply treats them as lacking suitable targets.

This fixes issue #970

(I do not have a ERC20 wallet)

Copy link

vercel bot commented Apr 23, 2025

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

Name Status Preview Updated (UTC)
ancientbeast ✅ Ready (Inspect) Visit Preview Apr 23, 2025 7:24pm

@DreadKnight
Copy link
Member

@Kevin4335 This breaks the game. Notice the builds around here.
Also referencing issues is done by using hashtag and issue number.

@Kevin4335
Copy link
Author

Locally, it passed the Jest tests and it seems to run.

@DreadKnight
Copy link
Member

Locally, it passed the Jest tests and it seems to run.

@Kevin4335 I bet, but you know how development goes when it comes to different machines... you could also poke at a browser based IDE, even Github provides one. Usually there's a small thing that breaks big changes on various set-ups.

@DreadKnight DreadKnight changed the title issue 970 showcase ability range on click fix Apr 26, 2025
@DreadKnight DreadKnight changed the title showcase ability range on click fix showcase ability range on click Apr 26, 2025
@DreadKnight
Copy link
Member

@Kevin4335 Really looking forward to test out this patch! 🤞🏻
No pressure, but note that if it takes too long, it might have some conflicts to resolve when trying to merge this into master 🐻

@DreadKnight
Copy link
Member

@Kevin4335 Any progress with this? Another week passed and this is starting to have conflicts...

@DreadKnight
Copy link
Member

@Kevin4335 Guess you went M.I.A... too bad, all this work will go to waste soon or it will take extra work, as there are PRs like this one that will get merged soon -> https://github.com/FreezingMoon/AncientBeast/pull/2733/files

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