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

Brown and pink aren't handled correctly in unifieddyes.get_color_from_dye_name #2

Open
wsor4035 opened this issue Dec 1, 2021 · 2 comments

Comments

@wsor4035
Copy link
Contributor

wsor4035 commented Dec 1, 2021

Brown and pink aren't handled correctly in unifieddyes.get_color_from_dye_name
For brown and pink dyes (alias medium_orange and light_red) no useful value is returned from unifieddyes.get_color_from_dye_name.

I'm not sure if this also affects other APIs, so I didn't create a Merge Request myself.

I'll append the correct hex codes, in case they are needed for an easier fix:

light_red: #ff5050
medium_orange: #b43500

@wsor4035
Copy link
Contributor Author

wsor4035 commented Dec 1, 2021

Vanessa Dannenberg @VanessaE · 1 year ago
Maintainer
Brown and pink have been kinda a thorn in my side since the advent of this mod. Their handling is inconsistent and not always logical... and that's mostly my fault 😛

The whole mod kinda needs an overhaul.

Edited by Vanessa Dannenberg 1 year ago

@wsor4035
Copy link
Contributor Author

wsor4035 commented Dec 1, 2021

SwissalpS @SwissalpS · 11 months ago
There are also duplicate hex values returned by that function for different colour names. Using this snippet and a linter on the output, you will get a list of duplicates.

Did I understand the system wrong, isn't every colour name supposed to identify a unique colour?

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

No branches or pull requests

1 participant