feat(tooltip): make tooltip border color overridable

[igera93]

On this pr I added the border color as overrideable property.
CLX ticket for the change requirement: https://instructure.atlassian.net/browse/CLX-233
Figma design with the new border color: https://www.figma.com/design/HJj54DmmepY3EWhVb8AYZ5/CEC-Design-System?node-id=1024-9289&node-type=frame&m=dev

Updated view for the Tooltip component:

Also I tried it out with red borders:

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[github-actions]

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-11-28 14:43 UTC

[balzss]

Hi @igera93 , Thanks for the PR! You can achieve this override without any modification in our codebase with something like this:

<InstUISettingsProvider
      theme={{
        themeOverrides: {
            componentOverrides: {
              Popover: {
                borderColor: 'yellow',
              }
            }
          }
      }}
    >

    <Tooltip
      renderTip="Hello. I'm a tool tip"
      as={Button}
      onShowContent={() => console.log('showing')}
      onHideContent={() => console.log('hidden')}
    >
      Hover or focus me
    </Tooltip>
</InstUISettingsProvider>

[igera93]

Hi @igera93 , Thanks for the PR! You can achieve this override without any modification in our codebase with something like this:

<InstUISettingsProvider
      theme={{
        themeOverrides: {
            componentOverrides: {
              Popover: {
                borderColor: 'yellow',
              }
            }
          }
      }}
    >

    <Tooltip
      renderTip="Hello. I'm a tool tip"
      as={Button}
      onShowContent={() => console.log('showing')}
      onHideContent={() => console.log('hidden')}
    >
      Hover or focus me
    </Tooltip>
</InstUISettingsProvider>

Oh nice, thanks for the heads up, at least I checked out the repo and tried out 😄 I will decline this pr 👍