Skip to content

Show reroll die prompt if move prompts are enabled#633

Open
sgreg wants to merge 1 commit into
iron-vault-plugin:mainfrom
sgreg:show-reroll-die-prompt
Open

Show reroll die prompt if move prompts are enabled#633
sgreg wants to merge 1 commit into
iron-vault-plugin:mainfrom
sgreg:show-reroll-die-prompt

Conversation

@sgreg
Copy link
Copy Markdown
Contributor

@sgreg sgreg commented Oct 15, 2025

If the "Prompt for rolls in Make a Move" setting is enabled, the "Reroll a die" action shows currently only a text field to enter the new die value. Adding now the same value selector as all other moves show when that setting is enabled, including the "Roll for me option".

Change the command to show a die value selector with a "Roll for me"
option to either set or reroll any of the action or challenge dice.
Also adds a "Cancel die" option, setting the die value to zero, for
example to cancel the action die on matching negative momentum.
@sgreg sgreg force-pushed the show-reroll-die-prompt branch from d76166b to 11f780a Compare May 23, 2026 12:55
@sgreg
Copy link
Copy Markdown
Contributor Author

sgreg commented May 23, 2026

Change of plans, as briefly hinted on Discord. Instead of showing the die value selector prompt when the setting is enabled, this PR now renames the command to "Reroll or set a die" and will always show the prompt.

There's a good bunch of assets that let you either set a die to a defined value (Ace, Archer, Armored, Chattering Skull, Loyalist, Shade, Weapon Master) or set the value from a pool of options (Musketeer, Pistoleer, Shapeshifter, Sleuth) where this change would IMO be handy.

And while at it, I added also a "Cancel die" option to the selector, which sets the value to zero. This lets you now manually cancel the action die on matched negative momentum, or cancel a challenge die (the Revenant asset allows that for example).

image




Also works when the challenge dice values were changed, e.g. to 1d8
image

@sgreg sgreg requested a review from zkat May 23, 2026 13:04
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