Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(tooltip): add dataIndex to valueFormatter callback #19123

Merged
merged 2 commits into from
Sep 26, 2023
Merged

feat(tooltip): add dataIndex to valueFormatter callback #19123

merged 2 commits into from
Sep 26, 2023

Conversation

ChepteaCatalin
Copy link
Contributor

@ChepteaCatalin ChepteaCatalin commented Sep 17, 2023

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

This PR adds dataIndex as the second parameter of the valueFormatter callback of the tooltip.

Fixed issues

Details

Before: What was the problem?

dataIndex was missing from valueFormatter callback of the tooltip but several library users needed it.

After: How does it behave after the fixing?

valueFormatter callback of the tooltip is receiving dataIndex parameter.

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#359

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 echarts-bot bot added PR: awaiting doc Document changes is required for this PR. PR: awaiting review labels Sep 17, 2023
@echarts-bot
Copy link

echarts-bot bot commented Sep 17, 2023

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.

Document changes are required in this PR. Please also make a PR to apache/echarts-doc for document changes and update the issue id in the PR description. When the doc PR is merged, the maintainers will remove the PR: awaiting doc label.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 20, 2023

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

@Ovilia Ovilia merged commit 80b251f into apache:master Sep 26, 2023
@echarts-bot
Copy link

echarts-bot bot commented Sep 26, 2023

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants