Manual updates to a counter value should impact effects without double-saving

[CrowbarToolsRobot]

Description

Currently when the "Save" button is clicked on the "Set Current Value" dialog after manually updating a counter's value, only the number that appears on the Edit Counter modal is changed as opposed to the counter's actual stored value (to update this, the user must also "Save" from the "Edit Counter" modal, which also closes out that modal). This means that updating the value and then testing "Effects On Update" runs those effects based on the old value rather than the one currently being prominently displayed in the modal (labeled "Current Value", largest text item on screen, user has already clicked a button labeled "Save" to set it, etc). To actually test the effects based on the "Current Value", the user must save and close BOTH modals, then re-open the exact same dialog and test the effects again. As a QoL feature it would make more sense for the counter's value to be truly updated as soon as the user clicks "Save" on the "Set Current Value" modal without also needing to save the "Edit Counter" modal and re-open it. Alternatively an in-line info message warning the user that the value they entered has not yet been saved would avoid the confusing experience of the UI showing the "Current Value" as one number whilst test effects actually reference a different value.

Submitted By

ollyrobz

Created via Discord.