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

Color issue 2. Naming of the colors from COLOR.md #77282

Closed
RedMisao opened this issue Oct 23, 2024 · 4 comments
Closed

Color issue 2. Naming of the colors from COLOR.md #77282

RedMisao opened this issue Oct 23, 2024 · 4 comments
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility

Comments

@RedMisao
Copy link
Contributor

Describe the bug

I'm not sure if the names of the colors are how they're intended to look, or the names are arbitrary and they look how they should.

Basically, it appears that colors have the wrong names.

The biggest offender is pink, which is definitely IRL magenta (#FF00FF). Even the rgb values say so (255,0,255 vs 254,0,254).
Directly related to this, ingame magenta is a shade of IRL purple (for reference, purple is this #800080 and it's different from violet, which looks like this #7F00FF).
Then, light_red is a kind of IRL pink.

The other offender is cyan, which is a kind of IRL light blue (teal), and light_cyan is actually how IRL cyan looks like (#00FFFF or (0,255,255 vs 0,240,255))

Finally, the grays, relatively minor compared to the previous two. There's light_gray and dark_gray, but not gray. In my opinion, dark_gray should be gray because there's a gap between that and black

Attach save file

N/A

Steps to reproduce

N/A

Expected behavior

Again, I'm not sure if the colors are intended to look how they currently do, or they're intended to look how they're named.

Screenshots

N/A, it can be seen in the docs.

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.19044.2965 (21H2)
  • Game Version: 7fc0414 [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

No response

@RedMisao RedMisao added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Oct 23, 2024
@Night-Pryanik
Copy link
Contributor

Naming of colors is such a deep pit.
fuschsia

IMG-20240913-WA0002
(Лиловый is purple in Russian)

@Procyonae
Copy link
Contributor

I think we just use the names that curses uses

@harakka
Copy link
Member

harakka commented Oct 23, 2024

Procyonae is correct, our colors and limitations on them stem from 16 color terminal where the program has zero control over what the colors actually are, it's up to terminal emulator.

@ZhilkinSerg
Copy link
Contributor

Actual RGB values are irrelevant - players can configure whatever they want in terminal settings and/or in SDL game builds (via color templates).

As for color ids: they were like that from the very beginning - https://github.com/Whales/Cataclysm/blob/69ffbb2953c1e92a9b24d5380dabe7f26066223e/color.h. There are pink instead of light_magenta, dark_gray and whatnot. Not much to gain in changing these ids.

@ZhilkinSerg ZhilkinSerg closed this as not planned Won't fix, can't repro, duplicate, stale Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility
Projects
None yet
Development

No branches or pull requests

5 participants