Wheel-adjustable batch size
Reviewed by: bb
Fixes #4767
Differential Revision: https://code.wildfiregames.com/D1034
Wheel-adjustable batch size
Description
Details
Event TimelineComment Actions (03:28:40) fpre: ctrl+shift+scroll batch training size to much possible by resources stock?
Comment Actions When changing the batchtrainsize in the options dialog in a running game, it is not reloaded anymore (until the user uses the moueswheel), while that was the case in alpha 22. You might want to add an options.json callback entry. Comment Actions A second concern is that the display of the batch count is only updated once per turn, whereas the value is updated instantly. Comment Actions Dunno, figuring out more details would imply coming up with the fix and then doing archeology on top of it. I just mentioned that here because it was the issue that prevented me from using this feature, because when I wanted to train 18 females with the mousewheel and then train 3 men afterwards, I had to wait 15 turns (7.5 seconds) to reduce it back to 3 with the mousewheel. There's probably just a missing update call in OnTrainMouseWheel (which would mean that if it was an issue with the click based batchtraining, then it was an issue in both a previous commit and in this commit?). Comment Actions That would require a quite big rewrite...
|