plotly.js icon indicating copy to clipboard operation
plotly.js copied to clipboard

feat: Switch color processing library

Open camdecoster opened this issue 4 months ago • 1 comments

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 avatar Aug 26 '25 15:08 camdecoster

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.

camdecoster avatar Oct 21 '25 00:10 camdecoster