Skip to content

Handle achromatic threshold better#462

Merged
facelessuser merged 1 commit intomainfrom
bugfix/better-achromatic-threshold
Mar 13, 2025
Merged

Handle achromatic threshold better#462
facelessuser merged 1 commit intomainfrom
bugfix/better-achromatic-threshold

Conversation

@facelessuser
Copy link
Copy Markdown
Owner

Spaces that are scaled on an order of magnitude of 1 should not have the same threshold of a space scaled at an order of magnitude of 100. If they are, this can cause colors to be considered achromatic much earlier if they are scaled small.

@gir-bot gir-bot added S: needs-review Needs to be reviewed and/or approved. C: docs Related to documentation. C: hsl HSL colors. C: hsv HSV colors. C: lab LAB colors. C: lch LCH colors. C: source Related to source code. C: tests Related to testing. labels Mar 13, 2025
Spaces that are scaled on an order of magnitude of 1 should not have the
same threshold of a space scaled at an order of magnitude of 100. If
they are, this can cause colors to be considered achromatic much
earlier if they are scaled small.
@facelessuser facelessuser force-pushed the bugfix/better-achromatic-threshold branch from 8745c40 to d0baaca Compare March 13, 2025 15:40
@facelessuser
Copy link
Copy Markdown
Owner Author

@gir-bot lgtm

@gir-bot gir-bot added S: approved The pull request is ready to be merged. and removed S: needs-review Needs to be reviewed and/or approved. labels Mar 13, 2025
@facelessuser facelessuser merged commit ff5851a into main Mar 13, 2025
16 checks passed
@facelessuser facelessuser deleted the bugfix/better-achromatic-threshold branch March 13, 2025 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C: docs Related to documentation. C: hsl HSL colors. C: hsv HSV colors. C: lab LAB colors. C: lch LCH colors. C: source Related to source code. C: tests Related to testing. S: approved The pull request is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants