Skip to content

fix(toolbox): only change title of requested feature. closes #12162#12168

Merged
pissang merged 1 commit intoapache:masterfrom
nsmgr8:toolbox-title-12162
Feb 19, 2020
Merged

fix(toolbox): only change title of requested feature. closes #12162#12168
pissang merged 1 commit intoapache:masterfrom
nsmgr8:toolbox-title-12162

Conversation

@nsmgr8
Copy link
Copy Markdown

@nsmgr8 nsmgr8 commented Feb 18, 2020

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Fixes the toolbox title issue described in #12162

Fixed issues

Details

Before: What was the problem?

After toggling magicType icons, titles in other features in toolbox become [object Object]. This is becaues the event fired a new title change and it was overwriting all features.

After: How is it fixed in this PR?

The event also sends which feature to change the title.

Usage

Are there any API changes?

  • The API has been changed.

Related test cases or examples to use the new APIs

NA.

Others

Merging options

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

Other information

@echarts-bot
Copy link
Copy Markdown

echarts-bot bot commented Feb 18, 2020

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.

@pissang
Copy link
Copy Markdown
Contributor

pissang commented Feb 19, 2020

Nice fix! Thanks!

@pissang pissang merged commit 331ac4a into apache:master Feb 19, 2020
@echarts-bot
Copy link
Copy Markdown

echarts-bot bot commented Feb 19, 2020

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

@nsmgr8
Copy link
Copy Markdown
Author

nsmgr8 commented Mar 21, 2020

May I know why this was not included in 4.7.0 release?

@sinpor
Copy link
Copy Markdown

sinpor commented May 26, 2020

@nsmgr8 The 4.8.0 has been resolved.🐛

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants