widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/bug-1395322-tool3 into lp:widelands

To: mp+290829@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Wed, 06 Apr 2016 10:49:15 -0000
In-reply-to: <20160403162723.12453.34610.launchpad@ackee.canonical.com>
Reply-to: mp+290829@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

I don't think that we should go to the trouble to remove the fallbacks - it would only add code. We document the intended keys, and if additional keys work too, I don't see a problem. Basically, if there is no third option, the behavour of the Ctrl key is undefined, so it doesn't matter if it switches to the 1st or 2nd tool.
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1395322-tool3/+merge/290829
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1395322-tool3.

References

[Merge] lp:~widelands-dev/widelands/bug-1395322-tool3 into lp:widelands
From: GunChleoc, 2016-04-03