Check for tooltip content before draw #4034
Conversation
src/core/core.tooltip.js
Outdated
|
|
||
| if (this._options.enabled) { | ||
| // Boolean for empty tooltip | ||
| var hasToolTipContent = Boolean(vm.title.length || vm.beforeBody.length || vm.body.length || vm.afterBody.length || vm.footer.length); |
There was a problem hiding this comment.
I'm not sure the explicit cast to a boolean is required here. Was this from your personal code style or am I missing something subtle?
There was a problem hiding this comment.
It's not required but I thought it made the code a little bit more readable. I can remove it if it doesn't
There was a problem hiding this comment.
Ok, lets leave it for now and see what @simonbrunel says. I have no real preference either way but I was leaning towards removing it just to keep the size down
There was a problem hiding this comment.
I agree with @etimberg, I would remove it because it's obvious that's a boolean (a || b || c).
There was a problem hiding this comment.
@etimberg @simonbrunel Okay sure thing, removed the boolean casting
|
@etimberg Is there a recommended way to monkey patch the tooltip |
|
@samueljo I think you can just override |
3 participants
Tooltips without any content (i.e. no
title,beforeBody,body,afterBody,footer) should not be drawn onto canvas as empty tooltips.Add check for tooltip content in
Core.Tooltip.draw.