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

enable chatlog in sp (GNA #9533) #1111

Closed
wesnoth-bugs opened this issue May 8, 2017 · 0 comments

Comments

@wesnoth-bugs
Copy link

@wesnoth-bugs wesnoth-bugs commented May 8, 2017

Original submission by anonymous on 2007-07-18

I know this may be a tall order, but it would probably eliminate lots of different bugs and misbehaviours that this separation continues to cause.

--

What, primarily, prompted me to file this report was the difficulty in getting the chatlog of a saved MP game to show.

  • If you "Load" the game (from the Single Player load game dialog), the chatlog isn't available. (There are other reports about differences between loading from a SP context vs. loading in a MP context.)

  • If you load the game as a "Local MP game", the chatlog isn't available.

  • So, you have to create a networked MP game to be able to view the chatlog.

(- See also the feature request / bug #4909 about the chatlog not being available in replay mode.)

--

The implications would be, e.g., that you could chat (to yourself, i.e. "for the record", i.e. for the replay) in SP mode, thus making notes in the chatlog about the gameplay.

(Reproduced on all)
Release: trunk
Priority: 5 - Normal
Severity: 2 - Minor

@gfgtdf gfgtdf changed the title Remove the separated logic between networked MP, local MP and SP gameplay (GNA #9533) enable chatlog in sp (GNA #9533) Apr 24, 2018
@sevu sevu added the Low-priority label May 14, 2019
@Wedge009 Wedge009 added the MP label Aug 21, 2019
@Wedge009 Wedge009 removed the MP label Sep 6, 2019
@jostephd jostephd added the Engine label Sep 15, 2019
@jostephd jostephd closed this in 2543fc6 Sep 15, 2019
jostephd added a commit that referenced this issue Sep 15, 2019
jostephd added a commit that referenced this issue Sep 17, 2019
jostephd added a commit that referenced this issue Sep 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.