widelands-dev team mailing list archive

Thread
Date

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

To: mp+349856@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Sun, 22 Jul 2018 16:55:39 -0000
In-reply-to: <153199376141.12517.16818035843454230137.launchpad@ackee.canonical.com>
Reply-to: mp+349856@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

You have done a beautiful job on those buildings! :)

There are 2 buildings that I'd still like to see changed in a follow-up branch though: The Port and the Tower. Their tower heights work really well as works of art, but they are obscuring the fields behind them, so it becomes a UI issue.

@bunnybot merge
-- 
https://code.launchpad.net/~widelands-dev/widelands/frisian-buildings/+merge/349856
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/frisian-buildings.

References

[Merge] lp:~widelands-dev/widelands/frisian-buildings into lp:widelands
From: Benedikt Straub, 2018-07-19