search

frappe / charts

Simple, responsive, modern SVG Charts with zero dependencies
https://frappe.io/charts
MIT License
14.9k stars 715 forks source link

Tooltips not rendering properly #223

Open kaapow opened 5 years ago

kaapow commented 5 years ago

Expected Behaviour

Tooltips should enclose all the data in a semi-transparent dialogue pop-up box.

Actual Behaviour

The left hand side of the dialogue box is chopped off. Not sure if this is a problem with some CSS coming from somewhere else in my code, or if there is something else I'm missing?

Steps to Reproduce:

I've created a simple test page to show how the demo code (copy and pasted directly): https://www.stet.build/test

NOTE: Add a GIF/Screenshot if required.

Frappé Charts version: 1.1.0

scmmishra commented 4 years ago

Hi @kaapow I was not able to replicate this issue, most likely this was fixed before I took over the project. Can you please upgrade to the latest release and see if you are still facing this.

There are no breaking changes between 1.1.X and 1.2.X, so the upgrading should not be an issue.

scmmishra commented 4 years ago

Hi @kaapow I was not able to replicate this issue, most likely this was fixed before I took over the project. Can you please upgrade to the latest release and see if you are still facing this.

There are no breaking changes between 1.1.X and 1.2.X, so the upgrading should not be an issue.

kaapow commented 4 years ago

Hi @scmmishra so I upgraded to 1.2.x and the problem persists. However what I did realise is that the problem is only there when the browser screen is wider than a phone size. In a portrait phone size configuration the pop-up renders correctly...so clearly it's a css thing. What I don't know if it's something I've added somewhere that somehow affects the frappe chart.

scmmishra commented 4 years ago

We use frappe charts extensively in our other products, as of now we haven't faced such an issue. If you have overridden any css classes, try removing them and see if the problem persists.