[Text] Fix optional id prop

[laurkim]

WHY are these changes introduced?

A recent PR that adds an id prop to the Text component was causing a TS error.

WHAT is this pull request doing?

Small fix to update the way the id prop is optionally rendered.

TS error

How to 🎩

🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines

Copy-paste this code in playground/Playground.tsx:

import React from 'react';

import {Page, Text} from '../src';

export function Playground() {
  return (
    <Page title="Playground">
      {/* Add the code you want to test in here */}
      <Text as="h1" variant="heading2xl">
        Text — no id
      </Text>
      <Text as="h1" variant="heading2xl" id="text-heading-test">
        Text — with id
      </Text>
    </Page>
  );
}

🎩 checklist

• Tested on mobile
• Tested on multiple browsers
• Tested for accessibility
• Updated the component's README.md with documentation changes
• Tophatted documentation changes in the style guide

[github-actions]

size-limit report 📦

Path	Size
polaris-react-cjs	209.13 KB (+0.01% 🔺)
polaris-react-esm	135.61 KB (+0.01% 🔺)
polaris-react-esnext	190.96 KB (+0.01% 🔺)
polaris-react-css	41.52 KB (0%)