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_matrix_saturation() applies a bright red layer to everything when used in the oversaturation client colour. #3355

Open
5 tasks done
MistakeNot4892 opened this issue Sep 20, 2023 · 0 comments
Labels
bug Something isn't working refactor artifact Secondary or unanticipated change deriving from broader system reworks.

Comments

@MistakeNot4892
Copy link
Contributor

Description of issue

/datum/client_color/oversaturated (applied when the druggy status is on a human mob) causes the game to become dark red. When color_matrix_saturation() is replaced with the previous colour saturation logic, the issue does not occur.

Difference between expected and actual behavior

Should be 1:1 with the legacy color saturation proc.

Steps to reproduce

Take a euphoriant drug.

Length of time in which bug has been known to occur

Likely since #2903

Client version, Server revision & Game ID

Current staging branch.

Issue bingo

  • Issue could be reproduced at least once
  • Issue could be reproduced by different players
  • Issue could be reproduced in multiple rounds
  • Issue happened in a recent (less than 7 days ago) round
  • Couldn't find an existing issue about this
@MistakeNot4892 MistakeNot4892 added bug Something isn't working refactor artifact Secondary or unanticipated change deriving from broader system reworks. labels Sep 20, 2023
@MistakeNot4892 MistakeNot4892 added this to the r3 milestone Sep 20, 2023
@MistakeNot4892 MistakeNot4892 removed this from the r3 milestone Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working refactor artifact Secondary or unanticipated change deriving from broader system reworks.
Projects
None yet
Development

No branches or pull requests

1 participant