Merge lp://staging/~widelands-dev/widelands/ctrl-priorities into lp://staging/widelands
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 8535 | ||||
Proposed branch: | lp://staging/~widelands-dev/widelands/ctrl-priorities | ||||
Merge into: | lp://staging/widelands | ||||
Diff against target: |
91 lines (+47/-2) 3 files modified
data/txts/tips/general_game.lua (+2/-2) src/wui/inputqueuedisplay.cc (+43/-0) src/wui/inputqueuedisplay.h (+2/-0) |
||||
To merge this branch: | bzr merge lp://staging/~widelands-dev/widelands/ctrl-priorities | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
GunChleoc | Approve | ||
Review via email:
|
Commit message
Updating all ware priorities of a building when CTRL is pressed while clicking.
Description of the change
For me it is a relatively frequent case that I want to change all ware priorities of a building at once, e.g., a low/high priority construction site. Currently this means clicking on multiple little red/green dots. This branch adds the possibility to hold the CTRL key while pressing one of the priority buttons to set all of them at once.
CTRL since the action more or less similar to the "dismantle the building without asking" function, kind of a strong request of some functionality.
Known bug: When a priority is CTRL-clicked for a ware that already has the priority set, the other wares are not updated. This happens since the Radiogroup does not relay the linked method when the value is already set. Could be changed but I didn't wanted to do so without asking since it will also affect other classes.
Feel free to reject this change. :-)
Continuous integration builds have changed state:
Travis build 2961. State: passed. Details: https:/ /travis- ci.org/ widelands/ widelands/ builds/ 317141860. /ci.appveyor. com/project/ widelands- dev/widelands/ build/_ widelands_ dev_widelands_ ctrl_priorities -2770.
Appveyor build 2770. State: success. Details: https:/