Skip to content

fix: preserve device colors for explicit palette entries - #610

Open
benletchford wants to merge 1 commit into
masterfrom
dev/civilization-ii-terrain
Open

fix: preserve device colors for explicit palette entries#610
benletchford wants to merge 1 commit into
masterfrom
dev/civilization-ii-terrain

Conversation

@benletchford

Copy link
Copy Markdown
Owner

Summary

  • keep the active device ColorTable entry unchanged whenever pmExplicit is present
  • apply explicit index semantics when usage also includes pmTolerant
  • update the activation regression test for mixed explicit usage

Validation

  • cargo fmt --all -- --check
  • cargo test activatepalette_preserves_device_rgb_for_mixed_explicit_entries
  • Civilization II startup dialogs, menus, and explored terrain recover their intended colors

References

@benletchford
benletchford marked this pull request as ready for review August 14, 2026 06:53
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.

1 participant