← Back to team overview

widelands-dev team mailing list archive

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

 

Thanks for taking this up, SirVer! 

Looks good to me on first sight (not extensively tested) with one exception: 
-- issue: the resource editing tool has to be deactivated while the RESOURCE VIEW is switched OFF. Deactivated means it must not be possible to create or remove any resources on the map.

Extended:
-- the 'q' key is targeted as a shortcut for the resource view button


-- 
https://code.launchpad.net/~widelands-dev/widelands/toggle_resources/+merge/328958
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/toggle_resources into lp:widelands.


References