LeagueSandbox / LeagueSandbox-Default

LeagueSandbox's Default Content Package
70 stars 97 forks source link

Add more map objects positions #69

Open danil179 opened 6 years ago

danil179 commented 6 years ago

Turret positions and level props missing.

add that from MapObjects file, turret positions also should be there in some cases.

MythicManiac commented 6 years ago

So how we really should go about this is to have Map and MapVariant content types.

Maps are the actual models and navgrids etc. a MapVariant belongs to a map and specifies things like spawn positions, objects on the map, etc.

This allows us to have multiple different MapVariants for the same Map, so if you want a variant where every player spawns on the center of the map and the edges are full of aggressive dragons for example, you can do that.

danil179 commented 6 years ago

MapObjects (.mob) file containing mostly level props and also some other things.

I'm not sure if it contains spawn position.

from what I know it contains:

  1. all level props minus shop
  2. turrets sometimes

so my issue is more specific about that.

and we should do variants also but I think this is another issue. (so after we include the info from mob file we edit it)

MythicManiac commented 6 years ago

I see map variants as a replacement for Riot's .mob file, and in terms of implementation it should be used over the current hardcoded values we have. It could be it's own issue to implement map variants in the first place if that's what you suggest.

We should regardless definitely extract the info from the mob file like you said.