[V4] Use Tooltip inside of a Popover

[jeroenransijn]

This PR implements the support of putting a Tooltip inside of a Popover.

Preview

Code Example

The above example is rendered by the following code.

<Popover content={<PopoverContentWithTextInput />}>
  <Tooltip content="Click me">
    <Button marginRight={20}>Tooltip Card + Popover</Button>
  </Tooltip>
</Popover>
<Popover content={<PopoverContentWithTextInput />}>
  <Tooltip
    appearance="card"
    content={
      <React.Fragment>
        <Heading>Heading</Heading>
        <Paragraph color="muted" marginTop={4}>
          Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do
          eiusmod tempor incididunt ut labore et dolore magna aliqua.
        </Paragraph>
      </React.Fragment>
    }
    statelessProps={{
      paddingY: 24,
      paddingX: 24,
      maxWidth: 280
    }}
  >
    <Button>Tooltip + Popover</Button>
  </Tooltip>
</Popover>

Other additions

This PR also adds the support for 2 appearances on a Tooltip: default and card.

[mshwery]

I'm wondering if this is a pattern we should even support (or encourage)? Curious about your thoughts on why we should pursue this @jeroenransijn ?

[mshwery]

Is it possible to not pass children? If so we may want to check that it exists first.

[mshwery]

This might be a misunderstanding of the semantics, but I've often followed the rule of thumb that a Tooltip is simple, short content. It cannot contain html or other components, only strings. With those semantics, things that look like a raised "card" would qualify as a Popover IMO.

Not sure if this meshes with Evergreen's component semantics, but thought it was worth mentioning here (if it allows us to simplify things).

[jeroenransijn]

The distinction currently between Tooltip and Popover is the interaction kind: hover vs. click. We might want to consider supporting something like <Popover interaction={InteractionKind.HOVER} /> similar to BlueprintJS at one point. I think we should leave that for a future major release.

[mshwery]

I would omit it from the propTypes if this is meant for internal use only.

[jeroenransijn]

It will throw a warning if it's not in propTypes right?

[mshwery]

Were these changes to ListItem intentional?

[jeroenransijn]

I am not sure why this is showing up, since this is already in v4.