[5.0] The state of `toolbox` will be changed to `normal` after the series leaves emphasis.

[plainheart]

Version

The current next branch

Steps to reproduce

Any cases with toolbox.
For example, test/toolbox-stack-custom.html

What is actually happening?

The state of the toolbox component was changed to normal.

What is expected?

The emphasis state of the toolbox component should be kept.

Others

I looked around and found it might be caused by the following code.
https://github.com/apache/incubator-echarts/blob/next/src/util/states.ts#L367-L378

According to the PR #12925, I think the blur state is supported in all series. But is it also supported in all components?
If not, perhaps we should only traverse the series and exclude the components like toolbox, axis etc..

Proposed changes

// Before
// model.eachComponent(function (componentType, componentModel) {
//     const view = componentType === 'series'
//         ? ecIns.getViewOfSeriesModel(componentModel as SeriesModel)
//         : ecIns.getViewOfComponentModel(componentModel);
//     // Leave blur anyway
//     view.group.traverse(function (child) {
//         singleLeaveBlur(child);
//     });
// });

// After
model.eachSeries(function (seriesModel) {
    const view = ecIns.getViewOfSeriesModel(seriesModel);
    // Leave blur anyway
    view.group.traverse(function (child) {
        singleLeaveBlur(child);
    });
});

I'm not sure, these are only my simple thoughts.
Thanks.

[echarts-bot]

Hi! We've received your issue and please be patient to get responded. 🎉
The average response time is expected to be within one day for weekdays.

In the meanwhile, please make sure that you have posted enough image to demo your request. You may also check out the API and chart option to get the answer.

If you don't get helped for a long time (over a week) or have an urgent question to ask, you may also send an email to dev@echarts.apache.org. Please attach the issue link if it's a technical questions.

If you are interested in the project, you may also subscribe our mail list.

Have a nice day! 🍵

[plainheart]

@pissang I've left some thoughts on this bug. Please help to check and fix it if possible. Thank you!

[pissang]

@plainheart Thanks for the detailed feedback. Some components like geo have blur and select states. So the code is to apply these states if component support. But toolbox is not supposed to be affected. I will look into it and fix it.

[plainheart]

It works well now. Thank you for the fix.