feat(gauge): progress.color supports 'auto'

[StNimmerlein]

Brief Information

This pull request is in the type of:

• bug fixing
• new feature
• others

What does this PR do?

Apply correct color to gauge chart progress when set to 'auto' in svg renderer.

Fixed issues

• fixes [Bug] Gauge progress 'auto' color not reflected when using SVG renderer #19884

Details

Before: What was the problem?

See the issue for more details.

After: How does it behave after the fixing?

Document Info

One of the following should be checked.

• This PR doesn't relate to document changes
• The document should be updated later
• The document changes have been made in apache/echarts-doc#xxx

Misc

ZRender Changes

• This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

N.A.

Others

Merging options

• Please squash the commits into a single one when merging.

Other information

[echarts-bot]

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

[Ovilia]

I think we should make this a new feature rather than bug fixing because 'auto' was not mentioned in the current doc. Please also make a pull request to the doc repo. Thanks!

[Ovilia]

This seems to be the same as in the above pointer. Let's create a autoColor after the const emphasisDisabled = emphasisModel.get('disabled'); line like:

const autoColor = (showPointer || showProgress)
    ? getColor(...)
    : null;

[StNimmerlein]

I extracted the variable, but without the ternary (so it could be used in other cases as well).
If you prefer the ternary to be there for performance purposes, I can add it.

[github-actions]

The changes brought by this PR can be previewed at: https://echarts.apache.org/examples/editor?version=PR-21224@0b06d7f

[StNimmerlein]

I think we should make this a new feature rather than bug fixing because 'auto' was not mentioned in the current doc. Please also make a pull request to the doc repo. Thanks!

In this section "auto" is mentioned for the pointer's color: https://github.com/apache/echarts-doc/blob/master/en/option/series/gauge.md#colorcolor--auto
Would it suffice to extend this paragraph by changing it to "The color of pointer and progress [...]"?

[Ovilia]

I think we should make this a new feature rather than bug fixing because 'auto' was not mentioned in the current doc. Please also make a pull request to the doc repo. Thanks!

In this section "auto" is mentioned for the pointer's color: https://github.com/apache/echarts-doc/blob/master/en/option/series/gauge.md#colorcolor--auto Would it suffice to extend this paragraph by changing it to "The color of pointer and progress [...]"?

Yes, please help copy and paste it to pointer and progress color. Thanks!

[echarts-bot]

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

[Ovilia]

Thanks for your contribution!