Skip to content

Codex/UI color palette#1502

Open
bodhibuurstede-sys wants to merge 3 commits into
RE-SS3D:developfrom
bodhibuurstede-sys:codex/ui-color-palette
Open

Codex/UI color palette#1502
bodhibuurstede-sys wants to merge 3 commits into
RE-SS3D:developfrom
bodhibuurstede-sys:codex/ui-color-palette

Conversation

@bodhibuurstede-sys
Copy link
Copy Markdown

Summary

PR checklist

  • The game builds properly without errors.
  • No unrelated changes are present.
  • No "trash" files are committed.
  • Relevant code is documented.
  • Update the related GitBook document, or create a new one if needed.

Pictures/Videos

Testing

Networking checklist

  • Works from host in host mode.
  • Works from server in server mode.
  • Works on server in client mode.
  • Works and is syncronized across different clients.
  • Is persistent.

Changes

Related issues/PRs

@bodhibuurstede-sys
Copy link
Copy Markdown
Author

Quick context for reviewers:

Summary

This PR adds a centralized UI color palette for shared SS3D interface colors.

It expands PaletteColors from a few loose static fields into an enum-backed palette that can be fetched by semantic key, enumerated by tools, and extended in one place. It also adds documentation for how to use the palette in UI code.

Changes

  • Added PaletteColor, PaletteColorDefinition, and lookup helpers to Assets/Scripts/SS3D/Data/PaletteColors.cs.
  • Added existing generic button overlay/text colors to the shared palette.
  • Kept the existing direct static color fields available for current callers.
  • Added Documents/UI_COLOR_PALETTE.md with usage guidance and extension steps.
  • Switched existing byte-style RGB palette values to Color32, since Unity's Color constructor expects normalized 0f..1f channels.

Testing

Not run locally. I do not have a Unity environment available in this Codex session, so this change was reviewed statically only.

Addresses #1377.

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