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

"has left the game" dialog says the player name, but not the side #2820

Closed
stevecotton opened this issue Apr 2, 2018 · 2 comments
Closed
Labels
Enhancement Issues that are requests for new features or changes to existing ones. MP Issues with multiplayer support or bundled multiplayer content. Stable 1.14 Issues specific to the Wesnoth 1.14 maintenance branch.
Milestone

Comments

@stevecotton
Copy link
Contributor

The behaviour of the dialog itself hasn't changed from 1.12.6, but the timing of the server chat messages has. This issue probably only matters where there's more than 2 players in the game.

Call the players "Host" and "Quitter", assuming Quitter had both side 2 and side 4.

On 1.12.6, Host sees this

  • chat message: Quitter has left the game
  • chat message: Host takes control of side 2
  • dialog for control/droid that says "Quitter has left the game", without saying which side
  • chat message: Host takes control of side 4
  • dialog for control/droid that says "Quitter has left the game", without saying which side

On 1.13.13, Host sees this

  • dialog for control/droid that says "Quitter has left the game", without saying which side
  • chat message: Quitter has left the game
  • chat message: Host takes control of side 2
  • chat message: Host takes control of side 4
  • dialog for control/droid that says "Quitter has left the game", without saying which side
@gfgtdf
Copy link
Contributor

gfgtdf commented Apr 2, 2018

hmm ye that dialgo shodul say for which side the side it's reassigned, bot for 1.14 we have a stringfreeze. But you coudl fix it for master then we coudl backport it for 1.14.1 .

I think it's save to assume that the host get the dialog for all side that that client controlled in order, and in most cases you probably want to do the same with all those sides anyways. So not a blocker for 1.14 i'd say.

@stevecotton
Copy link
Contributor Author

Agreed, not a blocker

@stevecotton stevecotton changed the title 1.13.13 "has left the game" dialog says the player name, but not the side "has left the game" dialog says the player name, but not the side Apr 2, 2018
@gfgtdf gfgtdf added Enhancement Issues that are requests for new features or changes to existing ones. MP Issues with multiplayer support or bundled multiplayer content. Stable 1.14 Issues specific to the Wesnoth 1.14 maintenance branch. labels Apr 23, 2018
@gfgtdf gfgtdf added this to the 1.14.1 milestone Apr 23, 2018
@gfgtdf gfgtdf closed this as completed in 06ebceb Apr 29, 2018
Vultraz pushed a commit that referenced this issue May 2, 2018
jostephd pushed a commit to jostephd/wesnoth that referenced this issue Oct 6, 2018
jostephd pushed a commit to jostephd/wesnoth that referenced this issue Oct 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Issues that are requests for new features or changes to existing ones. MP Issues with multiplayer support or bundled multiplayer content. Stable 1.14 Issues specific to the Wesnoth 1.14 maintenance branch.
Projects
None yet
Development

No branches or pull requests

2 participants