Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3627 closed defect (fixed)

[PATCH] Scenario player colors are replaced

Reported by: AlThePhoenix Owned by: elexis
Priority: Should Have Milestone: Alpha 20
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

Some custom player colours on maps are being replaced with the closest colour you could select in a random map or skirmish match. This doesn't make much sense for scenarios though (most likely an oversight).

Reproduce: Go into Atlas, change any player's colour to R 141 G 61 B 61 and save it as a scenario. When selecting that map in the gamesetup that player's colour will be black instead. This works with some other custom colours as well; I had another colour be replaced with green (I'm not sure how the replacement algorithm works).

Attachments (1)

t3627_dont_replace_scenario_colors_v1.patch (1.4 KB ) - added by elexis 8 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 by Stan, 8 years ago

This is as far as I can remember intended behaviour to prevent wrong colors. Now I agree with you that it should be fixed for scenarios.

comment:2 by elexis, 8 years ago

Keywords: patch review added
Milestone: BacklogAlpha 20
Summary: Some scenario player colours are being replaced in-game[PATCH] Scenario player colors are replaced

Refs #3431

comment:3 by elexis, 8 years ago

Owner: set to elexis
Resolution: fixed
Status: newclosed

In 17320:

Don't replace the custom colors of scenario maps with default colors. Fixes #3627.

comment:4 by elexis, 8 years ago

Keywords: review removed

Notice one can still change the colors in atlas for skirmish maps (which will be replaced with the closest matching default colors). This might/should be changed at some point. Same goes for starting resources and possibly other settings which can be changed in the gamesetup for skirmish maps.

Note: See TracTickets for help on using tickets.