Skip to content

Message box opens only for correct player#4842

Merged
Loobinex merged 1 commit into
masterfrom
reopenobjective
May 26, 2026
Merged

Message box opens only for correct player#4842
Loobinex merged 1 commit into
masterfrom
reopenobjective

Conversation

@Loobinex
Copy link
Copy Markdown
Member

Fixes #4840

@Loobinex Loobinex requested a review from rainlizard May 25, 2026 21:47
@rainlizard
Copy link
Copy Markdown
Contributor

rainlizard commented May 26, 2026

I need reproduction steps to review this. I've never seen this issue in multiplayer. Does it occur in singleplayer skirmish? Does this occur only in campaigns? Do you need special DKScript to display the objective? I don't know how that works.

I tested this fix with multiplayer and it's fine, but I don't think my test maps have objectives or something.

@PlebSupporter
Copy link
Copy Markdown

'Objective' is a type of message. It sits on the player hud as a green question mark, and they can read it at any time. For the most part the player will read it once and then dismiss it. When an active player dies, that objective windows shows itself again (when it shouldn't). It can cause confusion for players who think the objective has been updated, only to realize a few words or sentences in that they've already read that message before.

It happens everywhere. You can test by loading up any map with multiple keepers and killing one via the cheat menu. Simply having an objective on the map is enough.

You can quickly look at;
levels/standard/desperation
levels/deepdngn/korros-tor
campgns/keeporig/woodly-rhyme

@Loobinex
Copy link
Copy Markdown
Member Author

I need reproduction steps to review this. I've never seen this issue in multiplayer. Does it occur in singleplayer skirmish? Does this occur only in campaigns? Do you need special DKScript to display the objective? I don't know how that works.

I tested this fix with multiplayer and it's fine, but I don't think my test maps have objectives or something.

It happens in both multi and single player.

To reproduce:

  1. Have a map with an objective
  2. Minimize the objective
  3. Kill an enemy keeper
  4. Wait a few turns, until that keeper receives the 'you have lost' objective.
    -> Notice your minimized objective is restored.

Copy link
Copy Markdown
Contributor

@rainlizard rainlizard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see any issue, it seems to work.

@Loobinex Loobinex merged commit 9e49f21 into master May 26, 2026
2 checks passed
@Loobinex Loobinex deleted the reopenobjective branch May 26, 2026 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Objective message raises after any active keeper is defeated.

3 participants