-
Notifications
You must be signed in to change notification settings - Fork 19
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
Added clue replenishing to hotkey #771
Conversation
Chr1Z93
commented
Jul 14, 2024
- now properly places clue tokens that are spawned with numpad 8
- updated resource spawning for numpad 9
Two issues I ran into: (2) Adding more than ~16 clue tokens to a location causes all of the clues to vibrate and dance around on top of the card. Any more than 20 clues spawns them at the bottom left of the card instead. Perhaps we can convert these to clickable clue counters, just like the resource tokens? |
I don't think it should be intended to combine existing resource tokens with a clickable counter. Imo the existing resource token should not get deleted, because the counter takes precedence. (Should be easy to add that)
I'd like to avoid having clickable clue counters on locations since that will effect a lot of other code (like "Take a clue" hotkey and VP counting). How about spawning >16 clues on top of the existing ones instead? Here's something related: #390 |
I'm fine with your solution to (2), maybe wrapping up #390 in the process. For (1), as long as these are consistent, that's much better, but I'm also thinking about ways to merge the clickable resource counters and tokens that get spawned in other ways, not just a manually placed resource or charge. For example, dealing with the tokens (or the single clickable resource counter) that spawn with Extra Ammunition and the like. If you move those tokens over to a weapon that already has tons of tokens on it, it'd be nice to accommodate that without the extra mess. |