Code review comment for lp:~widelands-dev/widelands/rework-resource-indicators

Revision history for this message
GunChleoc (gunchleoc) wrote :

Code looks good to me - I have removed the iterators and pushed a commit.

- Barbarian indicators are good.
- Empire: I still have a bit of difficulty seeing the difference between no resources and stones. Maybe give them a bevel or something? Or give the marble a bit of color. The Arena's color might work better.
- Atlanteans: I still can't see the indicators for stones on a grey mountain. They need brighter colors overall. Lay the borders on thick.
- Frisians are good now

« Back to merge proposal