widelands-dev team mailing list archive

Thread
Date

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

To: mp+258719@xxxxxxxxxxxxxxxxxx
From: TiborB <tiborb95@xxxxxxxxx>
Date: Mon, 11 May 2015 08:29:10 -0000
In-reply-to: <20150510110321.17861.76859.launchpad@ackee.canonical.com>
Reply-to: mp+258719@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

The layout is bit inconsistent between wares vs. Workers&Buildings now, I mean vertical vs horizontal division.

Also I would add some space before sections - I mean above Lore, General, Dependencies.... to provide some visual separation.

But generally it works as supposed and looks good. Also code looks good to me.
-- 
https://code.launchpad.net/~widelands-dev/widelands/building_encyclopedia/+merge/258719
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/building_encyclopedia into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/building_encyclopedia into lp:widelands
From: GunChleoc, 2015-05-10