feat: Switch color processing library by camdecoster · Pull Request #7536 · plotly/plotly.js

Skip to content

Navigation Menu

Sign in

Appearance settings

Conversation

@camdecoster

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

@gvwilson gvwilson added P1

needed for current cycle

fix

fixes something broken

labels

Aug 28, 2025

@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.

Labels

fix

fixes something broken

P1

needed for current cycle

2 participants

@camdecoster @gvwilson