MrGadget NetworkManager populate spawnable prefabs button suggestions

[miwarnec]

posting so it doesn't get lost.

Few suggestions:

1. Do the OrderBy on the identities list before doing the AddRange, so that any that were added to NM.spawnable manually first are kept in place
2. Add a Clear button so users can wipe the list and rescan to rebuild the list fully sorted if they know that won't break them
3. make sure you're not adding Player Prefab to spawnables, since Mirror adds that automatically elsewhere.

For no. 1 above, I'm thinking of the case where player character prefabs are intentionally put in the first few slots and referenced by index in char selection, so they can't be sorted down into the middle of the list without breaking things

[miwarnec]

clear button:

[miwarnec]

as for OrderBy: this goes both ways:

• some users may prefer the original prefabs to stay unordered, followed by ordered
• otherwise may prefer the whole list to be ordered

I think it's more likely that people want everything to be ordered, since the Populate button is meant to add Everything.

[miwarnec]

#3915