Skip to content

Conversation

@camdecoster
Copy link
Contributor

@camdecoster camdecoster commented Aug 26, 2025

Description

Switch color processing library from TinyColor to color.

Closes #7523.

Changes

  • Change color processing library
    • hsv color strings are no longer permitted
    • Decimal fraction values of rgb strings are no longer permitted. This is a regression, but it's a shortcoming of the library and I'm working on getting it updated.
    • CSS named colors must be lowercase. For example, one must use 'blue' instead of 'Blue'. This is a regression, but it's a shortcoming of the library and I'm working on getting it updated.
  • Update/refactor method calls for dealing with colors
  • Update tests per library change

Testing

  • TBA

Notes

  • The new library allows for use of modern CSS color syntax

TODO

  • Add draftlog
  • Update attribute info about color syntax

@camdecoster camdecoster self-assigned this Aug 26, 2025
@gvwilson gvwilson added P1 needed for current cycle fix fixes something broken labels Aug 28, 2025
@camdecoster
Copy link
Contributor Author

I created a couple of PRs (#78 and #81) in a dependency used by color that will add some new features for parsing color strings that would bring it closer to parity with TinyColor. The maintainer recently merged one of these and will (hopefully) be merging the other soon. Once he publishes a new release, I'll be able to use it in plotly.js.

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

Labels

fix fixes something broken P1 needed for current cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update color processing library to handle modern CSS color syntax

2 participants