Tooltip formatter

haven't gotten to tooltips yet. I will make sure they're functional before version 1.0.

#4 works for me as well.

This module is nice work, but needs quite some work & cleanup to get all of it working. I'll look into submitting a series of patches.

This is resolved a little differently (for full highcharts API flexibility) in #1507800: Views-independent Highcharts API.

@stovak, if you agree on the approach there, we could mark this as duplicate and handle the tasks on that issue.

^ Specifically we solved this by explicitly adding the highchart options object methods separately, which couldn't be passed through Drupal.settings. Check out the bottom of highcharts_render() in http://drupal.org/sandbox/scottrigby/1507774. The tooltip is the only example added atm, as a proof-of-concept - and it works well without modifying anything else.

I am using 7.x-1.0-alpha6, but I cannot seem to get the tooltip working. I get the following error: D.call is not a function. I did try out the solutions outlined above. Help please!!!

I also have some problems with OG + UUID. After I installed uuid (because of highcharts), some OG views handlers just disappeared, I noticed that, OG relationship are broken and for example I can't use (group) OG group: OG membership.

Another thing is uuid_tokens it's also broken.

So for now at least for me uuid should be avoided if it’s possible.

Regardless UUID troubles, hightcharts is great module and I'm really grateful for it.

Thanks!

Does it still work when you change the function, so it is not the default? Because to me it seems that the whole $formatter piece of code doesn't do anything.

Anyone interested may want to help test the 7.x-2.x branch, where this issue has been resolved. The new API there has been tested including tooltip), but the Views integration in that branch has not been thoroughly tested yet.

I think @stovak's comment in #1561694-5: Highcharts views theme functions and templates was meant for this issue:

We're currently re-writing the core configuration of this module to make it more responsive and object-oriented. We'll have a solution for the tooltip formatter soon. One that makes sense and is consistant with drupal standards.
We're looking at a config stack that isn't based on loading a JSON file. Hope to have it committed soon.
-tom

In any case that will happen in the 2.x branch.

Clarifying title to prevent further duplicates (like #1528822: Tool Tip Formatter and #1564910: Tooltip formatter doesn't seem to be implemented).

Again a really cool module. We actively use it and I just updated the site to the new 2.x branch. (check out ClimbCheck ). But now I would like to edit (not append) the tooltip. Is this possible through a hack, temporary?

So with my new config module, my idea is to add the formatting function by adding a Drupal.behavior with one function then reference the behavior in the new config class. Long story short, yes, the hack is temporary.

-tom

#4 did work previously, but now when I comment it out it causes my browser to crash as soon as I hover over the chart. I'm not even sure how to troublehsoot that.

I found this browser crash issue in Chrome, but found the solution to be adding

$highcharts_config->tooltip->useHTML = true;

...to views_highcharts.theme.inc, right above the lines that are commented out.

Something like this:

     $highcharts_config->tooltip->useHTML = true;
//  $highcharts_config->tooltip->formatter = 'eval(function() {
//  return ' . $options['format']['tooltip_formatter'] . ';
//}  )';

Hope that helps.

I still can't figure out how to actually edit the tooltip format, though.

Thanks for that, was wondering why it was crashing - Intersting that no solution has been found for editing the format of the tooltips though

@dshields What version of highcharts module are you using? the 7.x-2.x branch does not have the eval() code in it:
http://drupalcode.org/project/highcharts.git/blob/refs/heads/7.x-2.x:/hi...
Please test this branch and let us know if you still run into issues, or need any help implementing the tooltip.

@jeremymcminn you can edit the tooltip.

See #7 above, which became the 7.x-2.x branch in #1507800: Views-independent Highcharts API

There are some more niceties we've been working on that still need to be pushed, and will probably do that soon.

We should also #1542336: Add version information to the project page to avoid confusion about branches…

Hi Scott

Thanks for that, will test these out and let you know

Jeremy

Thanks for the help. While I think the project I was working on has wrapped up, I'd really like to be able to draw on this module in the future so I'll continue to test.

So far, I"m running into the same issue as #1547090: Autoload files

How about removing any tooltip code from 7.x-1.x until it's ready?

We've evaluated 7.x-2.x but want to stick with 7.x-1.x for the time being. I'm re-tagging this issue as 7.x-1.x because it hasn't been fixed for this branch.

Tooltips support is not essential for us but we need to get rid of the following error.

Strict warning: Creating default object from empty value in template_preprocess_views_highcharts() (line 84 of /sites/all/modules/highcharts/theme/views_highcharts.theme.inc).

Commenting out the tooltips-related code in views_highcharts.theme.inc as suggested in #4 fixes this, see attached patch.

Re-rolled patch to work with git checkout of 7.x-1.x-dev (different then the 7.x-1.x download version, see http://drupal.org/node/1773498).

I have just committed a fix to the 7.x-2.x branch for tooltips:

d516a73

Fix tooltips by adding the options methods inline js to the footer. Also genericize this to support other options that expect callback functions, and include a new hook_highcharts_methods_alter() to allow other modules to add to this list

@larskleiner Thanks for the patch for the improperly named 7.x-1.x-dev branch, which needs to be cleaned up (one way or another). But let's follow up on that in the other issue. This issue will focus on fixing the 2.x branch for now.

This bug stills here on the current 7.x-1.0-alpha6 version but none of the previously submitted patches seem to apply correctly on it, so here's the patch for 7.x-1.0-alpha6