🐛 fix: incorrect rgb colors

[srj31]

Plotly treats decimal numbers for rgb value weirdly, where if rgb values are (0.9, 0, 0) it uses 0.9 as a percentage to display bright red

Fixes #279

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

[RichDom2185]

The change does need seem correspondent with the PR description; how does the fix for operation ordering solve the issue?

[martin-henz]

Well, the description is a bit strange. It's clear that the original implemention intended to scale the color values by a factor of 255. The original implementation assumed that ?? has higher precedence than *. However, ?? has lower precedence than *. So

pt.color[1] ?? 0 * 255;

is parsed like

pt.color[1] ?? (0 * 255);

which was obviously wrong because the color value isn't scaled at all. The PR fixes this bug.

[RichDom2185]

Well, the description is a bit strange.

Yeah, it is. But I agree, the operation ordering is definitely incorrect. I guess we can merge this first and reopen the issue if it still persists.

[srj31]

I wanted to explain why the error was not caught before when tested with different inputs. For e.g. pt.color[0] ?? 0 * 255 would return 0.9 which plotly would convert it to the 0...255 range value and it looked as if it worked correctly.

[martin-henz]

Ah, makes sense. Thanks!

[martin-henz]

I wanted to explain why the error was not caught before when tested with different inputs. For e.g. pt.color[0] ?? 0 * 255 would return 0.9 which plotly would convert it to the 0...255 range value and it looked as if it worked correctly.

It's fixed.