Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Handle overlapping tooltips #192

Merged
merged 5 commits into from
May 26, 2018
Merged

Conversation

aniwng
Copy link
Contributor

@aniwng aniwng commented May 25, 2018

Fixing a visibility issue when tooltips overlap. Tooltips should 'merge' to a single tooltip to allow visibility of data.

See #171

@NightCatSama
Copy link
Owner

NightCatSama commented May 25, 2018

wow, this is an amazing feature.

I have a few ideas before i merge it. If you're willing to help me, can to realize it, or I can do it later.

  • Provides a property (default: true) to control whether the merge tooltips In the case of overlapping.
  • Provides a property(Similar to the formatter, default: {value1} - {value2}) to customize merge rules.

The above is just my suggestion. I look forward to your feedback.

@aniwng
Copy link
Contributor Author

aniwng commented May 25, 2018

Those are good suggestions!

Would you be okay with merging what is done now and handle with those ideas later? I can help you with that. We do want to use the official released version rather than from my fork.

@NightCatSama NightCatSama merged commit 25f6062 into NightCatSama:master May 26, 2018
@NightCatSama
Copy link
Owner

I have completed the points mentioned yesterday. Version 2.7.0 has been released

I really appreciate your contribution.

@suavelizard
Copy link

Is there a way to trigger merge on load?

@moh1434
Copy link

moh1434 commented Oct 5, 2021

How to use this feature?
There is no example in tooltip documentation
I use version 3.2.14

@NightCatSama
Copy link
Owner

@moh1434 Reference #412

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants