widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/ctrl-priorities into lp:widelands

To: mp+335276@xxxxxxxxxxxxxxxxxx
From: Notabilis <notabilis27@xxxxxx>
Date: Sun, 17 Dec 2017 09:05:04 -0000
In-reply-to: <151337305957.1241.5402198793065522496.launchpad@ackee.canonical.com>
Reply-to: mp+335276@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Thanks for the review! Enjoy the new latest version with:
- Handling of already set priorities. Turned out the radiogroup already supported it but I didn't noticed before.
- Documentation of the new feature in the tips on the loading screens. The Ctrl-click for dismantle and destroy is also documented there, so I think that is the appropriate place.
- No more recursion. Did no harm but I didn't liked it.
-- 
https://code.launchpad.net/~widelands-dev/widelands/ctrl-priorities/+merge/335276
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ctrl-priorities into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/ctrl-priorities into lp:widelands
From: Notabilis, 2017-12-15