Remove line break HTML element from tooltipText

Why are we adding the line break element to meta for tooltipText?

if (hasMeta) {
  tooltipText += meta + '<br>';
}

I am currently implementing meta as HTML, so I simply format and style the tooltip contents with HTML and CSS. The extra line from the <br> is unnecessary. Is the line break necessary in other implementations?

if (hasMeta) {
  tooltipText += meta;
}

https://github.com/Globegitter/chartist-plugin-tooltip/blob/master/src/scripts/chartist-plugin-tooltip.js#L87

[tedbreen]

I suppose it depends on how much flexibility a user wants. For example, I only want to show my custom meta component in the tooltip and hide the value. The value is added by default though, which explains the line break:

if (value) {
  if (options.currency) {
    value = options.currency + value.replace(/(\d)(?=(\d{3})+(?:\.\d+)?$)/g, '$1,');
  }
  value = '<span class="chartist-tooltip-value">' + value + '</span>';
  tooltipText += value;
}

For now, I’ve just edited the source file by removing the line break and creating an option for displaying the value:

if (value) {
  if (options.currency) {
    value = options.currency + value.replace(/(\d)(?=(\d{3})+(?:\.\d+)?$)/g, '$1,');
  }
  value = '<span class="chartist-tooltip-value">' + value + '</span>';
  if (!options.doNotShowValue) tooltipText += value;
}

Result:

[tedbreen]

@robjac Wow, that’s odd. I have never encountered that problem. I am still on Chartist v 0.9.8. I’ll keep an eye out for this when I get around to upgrading.