Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed minimap showing fog of war everywhere when you joined a mp game without being a player #7094

Merged
merged 4 commits into from
Jun 8, 2022

Conversation

ofetios
Copy link
Contributor

@ofetios ofetios commented Jun 7, 2022

Spectator has 0 explored tiles if you join a mp game without being invited (without having your user id in the list of players).
This caused the minimap to show the fog of war everywhere

resolves #7088

@ofetios ofetios changed the title reveal tiles if spectator fixed minimap showing fog of war everywhere when you joined a mp game without being a player Jun 7, 2022
@yairm210
Copy link
Owner

yairm210 commented Jun 7, 2022

Simplify to the old condition plus && !isSpectator()

@yairm210 yairm210 merged commit 29c55cd into yairm210:master Jun 8, 2022
@ofetios ofetios deleted the spectatorMinimap branch June 10, 2022 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Minimap bug
2 participants