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

Add team chat for spectators #9639

Merged
merged 2 commits into from Dec 1, 2015
Merged

Add team chat for spectators #9639

merged 2 commits into from Dec 1, 2015

Conversation

abcdefg30
Copy link
Member

Closes #2062.

@deniz1a
Copy link
Contributor

deniz1a commented Oct 15, 2015

When a player loses and becomes spectator, will he have the same team chat indicator as he did during game? Because that could be confusing, team chat would no longer mean the same thing. How about calling it something like "Spectator chat"?

@abcdefg30
Copy link
Member Author

Unfortunately it doesn't do that, but it's also another issue, I think: #4772.

@pchote
Copy link
Member

pchote commented Oct 15, 2015

We should clearly distinguish between spectator and team chat, in that case.

@abcdefg30
Copy link
Member Author

Updated. Using (Spectators) instead of (Team) now.

@Mailaender
Copy link
Member

Works as promised. ✅

@Micr0Bit
Copy link
Member

does that mean , we want "defeated players" (therefor having fog and shroud turned off for him) , to still help their team by telling them in the teamchat what the enemy is doing ?

so nobody else can read that ?

@pchote
Copy link
Member

pchote commented Oct 17, 2015

If the existing (before this pr) behaviour does that, then I don't think we need hold this PR up over it. We should definitely switch players from team chat to observer chat at some point, though.

@Micr0Bit
Copy link
Member

true

@pchote
Copy link
Member

pchote commented Nov 22, 2015

The defeated team-mate case is best solved by #3210:

  • If a player is on a team with at least one active player, then use the team-spec mode (no vis controls, regular team chat)
  • If a player is not on a team, or everyone in their team is defeated then they all drop to the full spec mode (vis controls, spec chat).

@obrakmann
Copy link
Contributor

👍 works as advertised. As team-chat already doesn't work for dead players and cheating is thus not possible, I won't hold this up to require that dead players get switched to spec chat as well. #3210 indeed sounds like a better solution to that problem.

obrakmann added a commit that referenced this pull request Dec 1, 2015
Add team chat for spectators
@obrakmann obrakmann merged commit fd2bd7c into OpenRA:bleed Dec 1, 2015
@obrakmann
Copy link
Contributor

Changelog

@abcdefg30 abcdefg30 deleted the specchat branch December 2, 2015 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants