Skip to content

Conversation

sjcobb
Copy link

@sjcobb sjcobb commented Jun 26, 2025

Brief Information

This pull request is in the type of:

  • bug fixing
  • new feature
  • others

What does this PR do?

Add axisPointer.alwaysShowContent: true option to force axisPointer to show when charts are connected. A common case is for all time series charts on a dashboard to share an xAxis axisPointer to correlate across charts when hovering. Currently this does not work if series data is aligned differently / has null values, the axisPointer flickers instead of showing at the correct timestamp

Fixed issues

Details

Before: What was the problem?

Inconsistent display of axisPointer for charts using 'connect':

before_connect_always_fix.mov

After: How does it behave after the fixing?

axisPointer always shows on all connected charts:

after_connect_always_fix.mov

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 echarts-bot bot added the PR: awaiting doc Document changes is required for this PR. label Jun 26, 2025
Copy link

echarts-bot bot commented Jun 26, 2025

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.

@sjcobb sjcobb marked this pull request as ready for review June 26, 2025 19:13
Copy link
Contributor

@Ovilia Ovilia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider adding axisPointer.alwaysShowContent as tooltip.alwaysShowContent.

@sjcobb
Copy link
Author

sjcobb commented Aug 15, 2025

Consider adding axisPointer.alwaysShowContent as tooltip.alwaysShowContent.

Thanks for the feedback! This is fixed now to be axisPointer.alwaysShowContent

@sjcobb sjcobb force-pushed the fix-axis-pointer-connect branch from ad5f4f2 to bfcead5 Compare August 15, 2025 17:56
@sjcobb sjcobb requested a review from Ovilia August 15, 2025 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: awaiting doc Document changes is required for this PR. PR: awaiting review size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants