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

TRoW S22 Fixed the possibility of a missplaced dialogue when a bridge was broken #6430

Merged
merged 1 commit into from Jan 16, 2022

Conversation

Descacharrado
Copy link
Contributor

@Descacharrado Descacharrado commented Jan 10, 2022

New behaviour:
When no units are in the "island" north of the bridge, "retreat" is said instead of "advance".
advance
retreat

If there are units on both sides of the bridge, (even if it is only one, minor level 1 unit) the word advance takes priority over retreat. As stated on the issue, it is such a minor possibility that it does not matter if the condition is simple, as long as it does not feel weird when the dialogue is triggered (at least, that is my opinion).

Also remember string freeze if backported.

Closes #6376

@github-actions github-actions bot added the Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign label Jan 10, 2022
@Wedge009 Wedge009 added the Backport A reminder of a bugfix that was added to master that needs to be duplicated on the stable branch. label Jan 10, 2022
@Descacharrado Descacharrado changed the title TRoW S22 Fixed the possibility of a missplaces dialogue when a bridge was broken TRoW S22 Fixed the possibility of a missplaced dialogue when a bridge was broken Jan 12, 2022
@Wedge009 Wedge009 merged commit 68453aa into wesnoth:master Jan 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backport A reminder of a bugfix that was added to master that needs to be duplicated on the stable branch. Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TRoW S22 An event throws missplaced dialogue
3 participants