widelands-dev team mailing list archive

Thread
Date

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

To: mp+315330@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Mon, 23 Jan 2017 09:05:28 -0000
In-reply-to: <20170123082222.14871.50585.launchpad@ackee.canonical.com>
Reply-to: mp+315330@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

Code LGTM. Tested and this fixes the bug.

I am in favour of aborting the movement though as the bzr reporter suggested, and also of slowing down the movement. It is making me motion sick.

Let's wait for Travis then merge this one, because it's already an improvement.
-- 
https://code.launchpad.net/~widelands-dev/widelands/use_correct_zoom/+merge/315330
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/use_correct_zoom.

References

[Merge] lp:~widelands-dev/widelands/use_correct_zoom into lp:widelands
From: SirVer, 2017-01-23