Jump to content

Flare disabled for observers


Darkcity
 Share

Recommended Posts

My tendency would be to treat defeated players as observers and cut them off from their former allies - they can no longer access the team chat at that point either. However, they could still keep their color when flaring to observers - they also keep their color in the global chat.

  • Like 1
Link to comment
Share on other sites

To my knowledge, players can only be muted in the lobby chat, but not the pregame or ingame chats. I wonder why, though, since hosts can already kick and ban players - so, I imagine adding a mute command to be pretty straightforward.

15 hours ago, real_tabasco_sauce said:

If possible, it would be a nice touch to hide the observer flares of players you have muted.

I'd even go as as far and say this is a must, flares can be much more annoying than chat messages.

Link to comment
Share on other sites

15 minutes ago, Vantha said:

To my knowledge, players can only be muted in the lobby chat, but not the pregame or ingame chats. I wonder why, though, since hosts can already kick and ban players - so, I imagine adding a mute command to be pretty straightforward.

I'd even go as as far and say this is a must, flares can be much more annoying than chat messages.

See https://code.wildfiregames.com/D4871

And @Itms rewrite here https://gitlab.com/na-Itms/0ad/-/compare/master...moderation?from_project_id=188465

 

  • Thanks 1
Link to comment
Share on other sites

I'd like to submit the code now, but it comprises of two split features that should probably get a pull request of their own. Firstly, the chat notifications, and secondly (built on top of it) the observer flares. They are very interconnected and I would like to have both pull requests open at the same time (especially since the chat notifications would become kinda pointless should the observer flare code be rejected).

How should I do that? Open a pull request on the main 0ad repo for the base code (chat notifications) and then open another one to that new branch on my fork containing the additional code (observer flares)?

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...