Code review comment for lp:~widelands-dev/widelands/economy-target-profiles

Revision history for this message
Toni Förster (stonerl) wrote :

Very good so far, two nits though.

Stockpile and Efficiency Profiles should be changeable but not deletable. Also, when deleting the active profile it should switch back to default. Otherwise, the deleted one is still shown as active and the drop-down list has an empty entry.

« Back to merge proposal