-
Notifications
You must be signed in to change notification settings - Fork 43
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
Citizens management needs more increment buttons #50
Comments
Agree. One solution: Let user choose which buttons to show? Have a list of buttons in settings, check the buttons you want to use. |
I added 1k to my fork, but this is still an issue. |
Agree as well, very inconvenient. Another solution option - instead of the +/- ∞ buttons should be +/- custom. Custom value would be a field on each line so each "category" has it's own custom value (I move differing amounts of each type - Tanners and Smiths by the thousand. Miners and Farmers by the 100k or 500k, Wood workers by 50 or 100k, Soldiers by 500k and Cavalry by 50k. This same scheme should work and be applied imho to the Buildings tab as well. In my view the buttons should be +1 +10 +100 +Custom |
Does this repo accept pull requests? |
@olleharstedt -- As long as the PR is focused and doesn't radically alter the game, then yes. I don't check this repo too often though, so don't expect anything fast. 😅 |
Possible duplicate of #33 ? |
Maybe we can add a project page in Phabricator or such, to discuss possible directions? |
I'm not sure what Phabricator is. Discussion here on the issues works well enough for me. |
Currently citizens management is capped at +/-100 after which it goes to +∞. Would be nice to have increments such as +1k and such when available, as is possible for buildings. Gets pretty tedious at the moment after a while 😄
The "Custom increment" option is there but it also removes all the other buttons which ends up being inconvenient when frequently having to switch between 100/1K/10K for different types of units/buildings
The text was updated successfully, but these errors were encountered: