landenf / Settlers-of-Catan

Multiplayer Online version of Settlers of Catan
https://catan-4dea2.web.app
2 stars 0 forks source link

#78 create settlements #88

Closed landenf closed 2 months ago

landenf commented 2 months ago

Should be good to go the settlements will appear if the level is greater than 0 in the community spaces dictionary.

landenf commented 2 months ago

@SirFatredVIII lets look in mtg tom this is what it looks like on my screen image

landenf commented 2 months ago

Team, After reviewing the currently implemented code should be ok for what we are trying to do. Because we are keeping a dictionary of all tiles and their respective settlements, and for purposes of validating where settlements are allowed to be placed, we need to include a settlement in all respective tiles dictionary's. When doing so, we will for sure 'paint' the settlement on top of everything no matter what tile is rendered first.

For example: There is a settlement displayed here: image And thus will be included in tile 5's 3 position, tile 10's 5 position, as well as tile 6's 0 position. Leaving a dictionary looking like so: image This was created for testing purposes. With that, and no further concerns, I believe our UI for settlements is complete.