Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/fh1-finish_help into lp:widelands

2018-01-05 Thread GunChleoc
1. It as always been right aligned - I don't want to make a design change in 
this branch

2. We have a maximum limit of images per row, so it won't get too wide on 
smaller resolutions. So, also a proposed design change and not a bug in this 
branch.

Thanks for the review! :)

It was already designed this way before I ever started working on it, and I 
wouldn't object to doing a redesign some time in the future. It would be really 
nice to have some proper tables here.


@bunnybot merge
-- 
https://code.launchpad.net/~widelands-dev/widelands/fh1-finish_help/+merge/335708
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/fh1-finish_help into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp


Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/fh1-finish_help into lp:widelands

2018-01-04 Thread kaputtnik
Found two things:

1. In 'Workers' and 'Buildings' are some headers called 'Experience levels' and 
the text of the next line saying 'Miner to Master (19EP)' is right aligned. All 
other texts are left aligned, so this should also be left aligned. Found for 
'Miners' and '*mines' of empire

2. In 'Buildings' -> 'Cumulative cost' the images are mostly in the same line 
as the string. But starting from 7 Images there is a line break showing some 
few wares in first row and the other 6 in the next row. E.g. Empire Colloseum 
shows:

One row: '7 x Planks'  one image
next row:  6 images

All other looks good to me :-)
-- 
https://code.launchpad.net/~widelands-dev/widelands/fh1-finish_help/+merge/335708
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/fh1-finish_help into lp:widelands.

___
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp