Make hcTooltip support conditional expressions

[isaaclyman]

Describe your idea

It would be cool if [hcTooltip] could be conditionally rendered.

What problem are you trying to solve?

We have situations where a tooltip is needed in some states but not others. It's natural to try to do:

[hcTooltip]="shouldShowTooltip ? 'You can't do that right now.' : null"

Or

[hcTooltip]="shouldShowTooltip ? 'You can't do that right now.' : undefined"

But in both cases, an empty tooltip is still shown (see below). It would be great if the tooltip only showed when there was content in it.

Additional context

[bskeen]

Note: This is possible currently by binding to the [trigger] property on the HcPopoverAnchorDirective (that is how it is handled in the Instance Switcher in Cashmere, see below).

Fabric.Cashmere/projects/cashmere/src/lib/instance-switcher/instance-switcher.component.html

Lines 64 to 65 in f0c1ddd

	[hcTooltip]="_tooltipText.addText"
	[trigger]="_tooltipText.addTrigger"

Fabric.Cashmere/projects/cashmere/src/lib/instance-switcher/instance-switcher.component.ts

Lines 176 to 179 in f0c1ddd

	this._tooltipText = {
	...toolTipVal,
	addTrigger: toolTipVal.addText ? 'hover' : 'none',
	closeTrigger: toolTipVal.closeText ? 'hover' : 'none'

[isaaclyman]

Oh, brilliant. I'll use that as a workaround for now.

[andrew-frueh]

Yup, you nailed it @bskeen - using trigger that way is exactly how I do conditional tooltips or popovers. I do like your suggestion though @isaaclyman - would be cleaner code if you could get the same functionality with one line rather than having to set it in two places.

[health-catalyst]

🎉 This issue has been resolved in version 14.1.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀