apache / echarts

Apache ECharts is a powerful, interactive charting and data visualization library for browser
https://echarts.apache.org
Apache License 2.0
60.75k stars 19.62k forks source link

tooltip.appendToBody scroll-issue on touch devices #12776

Open tmtron opened 4 years ago

tmtron commented 4 years ago

Version

4.8.0

Reproduction link

https://stackblitz.com/edit/angular-echarts-append-to-body

Steps to reproduce

2020-06-09_08h50_55

What is expected?

When some e-charts are in a scroll-container the tooltip should scroll (or maybe be hidden, if scroll is not possible)

What is actually happening?

When some e-charts are in a scroll-container the tooltip will remain fixed (to the body)..

Additional

On a real phone (using Goolge Chrome mobile), we see an additional issue (which may have the same cause).
When we scroll down to the 2nd or 3rd echart and click a bar the tooltip will not show up (I guess that the calculated position is off the screen boundaries.

echarts-bot[bot] commented 4 years ago

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! 🍵

wf123537200 commented 4 years ago

Is the issue the same as #12778 ?

tmtron commented 4 years ago

@Ovilia why the waiting-for:author tag: do you need some more info?

ArturMiszkowicz commented 3 years ago

This also persist in version 5.0.0. (Reproduce: https://jsfiddle.net/zndot4ub/). As you can see on screenshot (anonymyzed) there are two tooltips, while mouse is between charts (during scroll). The tooltips will be there until user moves his mouse over chart to show another tooltip (or move on canvas where chart is rendered).

image

sunbin12121 commented 3 years ago

加 1

pissang commented 3 years ago

@tmtron It should be the same with #12778. Which has been fixed in 5.0 @ArturMiszkowicz I think it's a new issue brought in 5.0.0. It has been fixed since 5.0.1 https://github.com/apache/echarts/releases/tag/5.0.1

tmtron commented 3 years ago

@pissang I can still reproduce the issue in 5.0.1

Here is an updated Stackblitz example

plainheart commented 3 years ago

Yes, this issue seems to be there. I'm wondering how we should properly handle this case, hide or scroll?

tmtron commented 3 years ago

Yes, this issue seems to be there. I'm wondering how we should properly handle this case, hide or scroll?

I could live with both - maybe it makes sense to add an option and let the users choose.

Flourad commented 3 years ago

移动端appendToBody开启后tooltip无法正常关闭的bug在最新版本里还是存在,请问什么时候可以解决

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale because it did not have recent activity. It will be closed in 7 days if no further activity occurs. If you wish not to mark it as stale, please leave a comment in this issue.

tmtron commented 1 year ago

+1

programmingPug commented 1 year ago

Still an issue, can we close the tooltip on scroll?

programmingPug commented 1 year ago

Actually found a solution with my own question, on the parent of the scrollable container have a listener for scroll then just call the method and dispatch the hide action.

public onScroll(event: Event): void { this.chart.dispatchAction({ type: 'hideTip' }); }

xy1041 commented 2 months ago

Still an issue, any updates?