Heading: Nested icons inherit text colour

[michaeltaranto]

When using icons inside of a Heading, the default tone was always neutral, rather than inheriting the colour of the nearest component.

For example, when an icon was used inside of a TextLink within a Heading:

<Heading level="1">
  Title with <TextLink>link <IconArrow /></TextLink>
</Heading>
// => Previously, IconArrow was the heading text colour
// `neutral`, now inherits the `link` colour.

or equally, when an icon was used inside of a Secondary component within a Heading:

<Heading level="1">
  Title with <Secondary>secondary <IconArrow /></Secondary>
</Heading>
// => Previously, IconArrow was the heading text colour
// `neutral`, now inherits the `secondary` colour.

[changeset-bot]

🦋 Changeset detected

Latest commit: c4a7694

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
braid-design-system	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[mrm007]

Massive cleanup 👏

Just a few semi-nitpicky comments from me 👼