← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/rework-resource-indicators into lp:widelands

 

Implemented most of your comments, and changed the graphics. Didn´t touch the atlantean images yet, will do so now.

I disagree about hardcoding the none-resi as "none". Hardcoded name restrictions are always a bad thing; it should be possible for a future modder to create a new resource named "none" if they want. Better not to have any hardcoded names and use the only string that makes no sense at all as a resource name, "".
-- 
https://code.launchpad.net/~widelands-dev/widelands/rework-resource-indicators/+merge/353996
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/rework-resource-indicators.


References