BUG: Custom tooltip not rendering background

[musmuris]

When implementing a custom tootltip, the background is only under the title and not the title and the body.

Example at https://codepen.io/anon/pen/zyMRdG

Expected that the background should be covering the whole tootip

The options section is as follows:

options: {
  tooltips: {  		
    custom : t => {
      if(t.labelTextColors) {
        t.titleFontColor = t.labelTextColors[0];         
      }    		      
      t.title = ['Hello'];
      t.body  = [{ lines: ['foo']}];
    }
  }
}

My reason for a custom one is to override the title color depending on underlying data. A workaround is just to set the color like above but then implement the title and label callbacks, but it should be possible to just update the custom tooltip model in one go I thought?

[etimberg]

@musmuris unfortunately the custom tooltip option is meant for rendering the tooltip as HTML instead of on the canvas. I realize the name is confusing but we're stuck with it until v3.0.

We've been working on a number of PRs recently for "scriptable" options that support a function (see #5973, #5976, etc). I think we could support those here so ultimately what would happen is that you would set titleFontColor to a function that returns based on what's selected.

CC @simonbrunel @nagix @kurkle thoughts?

[etimberg]

@kurkle @benmccann thoughts on this?

My thoughts are:

• Update tootip fonts to be scriptable (title & footer get all items, body gets context per item in tooltip) which eliminates the need for using custom tooltips in this use case
• Consider renaming custom to something else (or drop entirely in favour of an HTML tooltip plugin) Advantage of the plugin method is ease of integration with other frameworks like react.

[LeeLenaleee]

Dont know how much work it is to make it into a plugin, but I think that renaming it to html or external from custom might be a good idea to make it more clear its not customization for the default tooltip.

[etimberg]

@kurkle thoughts on the above rename?

[kurkle]

Sounds good!

[etimberg]

Closing since custom was renamed to external and released in 3.0.0-beta.12