Code review comment for lp:~widelands-dev/widelands/ferry
Revision history for this message
| GunChleoc (gunchleoc) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
Barbarian Ferry Yard needs cloth to build, but there is no Weaving Mill, because that building needs seafaring.
It would be nice to have some indicators for the range when building a new waterway - I think we best wait with that until after we have the new work areas, because there could be some common code there. It would also be nice to have visual indicators for established waterways as an overlay.
Can you please remove the ferry animations except for the first frame each? They cause the diff to be truncated, because there are so many of them. Since they are just placeholders, 1 frame should be good enough for visual quality.
Code review of the bits that I can see is done.