youcef92 / flotr

Automatically exported from code.google.com/p/flotr
0 stars 0 forks source link

Tool Tip Pie charts #121

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Hi

I need to draw tool tips on pie charts and bar charts. I can draw the tool
tips for bar charts at the correct position, but not for pie charts. I can
see the tool tip but it is too far for some of the pie that i draw. How can
i track the correct position of the mouse track point. I am using the
attachec flotr.js and the function drawHit creates the mouse track point.

Please Help!

Original issue reported on code.google.com by sumeenk...@yahoo.com on 29 Mar 2010 at 5:56

Attachments:

GoogleCodeExporter commented 8 years ago
This is because my xaxis and yaxis scale is starting from -1.00, how can i make 
them
start from 0.0

Original comment by sumeenk...@yahoo.com on 29 Mar 2010 at 8:10

GoogleCodeExporter commented 8 years ago
Try providing xaxis: { min: 0 } in the options object.

Original comment by peter....@solide-ict.nl on 21 Jul 2010 at 12:02

GoogleCodeExporter commented 8 years ago
I tried that option but the problem is that the data that i return for pie 
chartsis json data points(for example - 
{"series":[{"data":[[0,"12"]],"label":"CPAP","seriesIndex":0,"bid":"37","perc":0
.54545454545455},{"data":[[0,"6"]],"label":"NPSG","seriesIndex":4,"bid":"37","pe
rc":0.27272727272727},{"data":[[0,4]],"label":"Other","seriesIndex":8,"perc":0.1
8181818181818,"bid":"37"}]}

If i use the scale, the tool tip is positioned correctly but the pie for that 
element is far from the tool tip. This is the problem for all my pie charts and 
 so i can not set any specific x:axis or y:axis number

Please see the attached file for the graph

Original comment by sumeenk...@yahoo.com on 21 Jul 2010 at 2:38

Attachments:

GoogleCodeExporter commented 8 years ago
Thanks for the graphics, they cleared up a lot (but next time, PLEASE don't put 
images in a docx file; they're a headache for me to open. Just send the 
individual images or put them in a tar/zip file if you have to)

I'm not sure but probably Fabien will know: Fabien, are tooltips for pie graphs 
(where the tooltip shows up near the mouse when you hover over a pie slice) 
supported at all?

Original comment by peter....@solide-ict.nl on 21 Jul 2010 at 3:12

GoogleCodeExporter commented 8 years ago
After reading a code, i'm sure that it isn't supported yet. But it could be 
easy done, like for a bars. But trigonometry need to by used:)

Original comment by macku30@gmail.com on 27 Jul 2010 at 6:19

GoogleCodeExporter commented 8 years ago
This is solution. It works only for 2D pies. It has to be set pie.show in 
global settings to works properly. You should make a code review, cose i just 
learned prototype and it could be not optimal.

Original comment by macku30@gmail.com on 27 Jul 2010 at 6:57

Attachments:

GoogleCodeExporter commented 8 years ago
macku30: Thank you for the contribution, but I have a few comments.

First, could you please attach a diff instead? A diff makes it easier to apply 
your changes if any modifications were made in the meanwhile you were making 
your changes (or in the time it takes to review this code and actual 
application of it)

Also, a diff against the latest subversion trunk would be more useful; the 
current file appears to be based on the 0.2.0alpha release, which is very old. 
Many changes have been made since this release so it would be hard to try and 
port your changes to the latest trunk revision.

Original comment by peter....@solide-ict.nl on 27 Jul 2010 at 7:09

GoogleCodeExporter commented 8 years ago
Ok i will try to make a diff, i based on rev 158 or something like this

Original comment by macku30@gmail.com on 27 Jul 2010 at 7:12

GoogleCodeExporter commented 8 years ago
Here is diff with r.168. My changes are mainly in lines beetwen 2000 and 3000 
so others are from diferent base files.

Original comment by macku30@gmail.com on 27 Jul 2010 at 8:29

Attachments:

GoogleCodeExporter commented 8 years ago
What I meant with a diff against the latest trunk was a diff containing only 
your changes against trunk.

This diff contains everything that has changed between the 0.2.0alpha release 
and trunk, plus your changes on top of that! I'm sure you understand we can't 
use that.

Original comment by peter....@solide-ict.nl on 27 Jul 2010 at 8:34

GoogleCodeExporter commented 8 years ago
ok, it was my mistake. Here it is

Original comment by macku30@gmail.com on 27 Jul 2010 at 9:47

Attachments:

GoogleCodeExporter commented 8 years ago
Great work, thanks a lot! The OP could perhaps try this out and let us know if 
this solves his problem.

Then Fabien can commit it (if he's still reading these messages...)

Original comment by peter....@solide-ict.nl on 27 Jul 2010 at 9:52

GoogleCodeExporter commented 8 years ago
Of course I still read the messages, that's just I don't have time to answser. 
I'll try the patch, thank you !

Original comment by fabien.menager on 27 Jul 2010 at 4:06

GoogleCodeExporter commented 8 years ago
awesome! U r great. It really fixed my problem. thank u so much.

Original comment by sumeenk...@yahoo.com on 27 Jul 2010 at 7:16

GoogleCodeExporter commented 8 years ago

Original comment by fabien.menager on 12 Aug 2010 at 12:52

GoogleCodeExporter commented 8 years ago
Shouldn't this be committed and the ticket closed? It solved the OP's problem.

Original comment by peter....@solide-ict.nl on 12 Aug 2010 at 2:09

GoogleCodeExporter commented 8 years ago
I need to review the code before commiting, I saw a few things that don't 
follow the project's coding style and rules.

Original comment by fabien.menager on 12 Aug 2010 at 2:23

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r171.

Original comment by fabien.menager on 8 Sep 2010 at 5:22