Code review comment for lp:~widelands-dev/widelands/peaceful

Revision history for this message
GunChleoc (gunchleoc) wrote :

Tested and working.

I think we should not allow peaceful mode for territorial time, territorial lord or autocrat, because it can make those game modes unwinnable. UI-wise, you could hook into the win condition dropdown's selected signal and show/hide the checkbox.

« Back to merge proposal