Changes between Version 1 and Version 2 of Ticket #2963, comment 1
- Timestamp:
- Dec 24, 2014, 10:31:44 PM (9 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #2963, comment 1
v1 v2 1 Also, things like the AISeed shouldn't be persisted.2 Perhaps it would be the best to manually remove these fields from the object.1 1) Also, things like the AISeed shouldn't be persisted. 2 2) Perhaps it would be the best to manually remove these fields from the object. 3 3 Then, the object had to be cloned though. 4 4 5 I don't know whether cancelSetup() in gamesetup.js gets called for multiplayer lobbies, too. 5 3) I don't know whether cancelSetup() in gamesetup.js gets called for multiplayer lobbies, too. 6 7 4) Would it make even more sense to split up into single/multiplayer-settings? 8 9 5) Just checked the behaviour of scenarios where civs are fixed, then after reloading, this setting confuses the UI logic etc. - perhaps omitting the entire player settings part from the settings might help there a lot - or just persist AI difficulties.