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

Adventure weighs hue too heavily when comparing colors #1077

Open
jtJava opened this issue May 22, 2024 · 3 comments
Open

Adventure weighs hue too heavily when comparing colors #1077

jtJava opened this issue May 22, 2024 · 3 comments

Comments

@jtJava
Copy link

jtJava commented May 22, 2024

static float distance(final @NotNull HSVLike self, final @NotNull HSVLike other) {

The distance method when comparing two colors causes a color like #f6fff5 (white with very slight green tint) to be downsampled to green instead of white.

@zml2008
Copy link
Member

zml2008 commented May 22, 2024

tbh not a priority for us to look into, since downsampling is rarely used outside of super legacy situations, but PRs are welcome (with visual comparisons)

@jtJava
Copy link
Author

jtJava commented May 22, 2024

I'm writing code for the latest version of Paper, I'd like to make use of modern colors while still allowing legacy clients to be able to play my server without seeing god awful color translation.

What do you need to see in the pull request? Is this not as simple as just removing the multiplication by three?

@zml2008
Copy link
Member

zml2008 commented May 22, 2024

legacy clients are at the bottom of our priority list these days, it's 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants