Implement code connect for segmentedControl and heading

[lukasoppermann]

Is this the correct code output?

Also @joshblack I think the SegmentedContorl has the same issue with leadingIcon where it does only accept a function but not an element. Can we fix that?

[changeset-bot]

⚠️ No Changeset found

Latest commit: 995d29f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

[github-actions]

Uh oh! @lukasoppermann, the image you shared is missing helpful alt text. Check your pull request body.

Alt text is an invisible description that helps screen readers describe images to blind or low-vision users. If you are using markdown to display images, add your alt text inside the brackets of the markdown image.

Learn more about alt text at Basic writing and formatting syntax: images on GitHub Docs.

🤖 Beep boop! This comment was added automatically by github/accessibility-alt-text-bot.

[github-actions]

Uh oh! @lukasoppermann, the image you shared is missing helpful alt text. Check your pull request body.

Alt text is an invisible description that helps screen readers describe images to blind or low-vision users. If you are using markdown to display images, add your alt text inside the brackets of the markdown image.

Learn more about alt text at Basic writing and formatting syntax: images on GitHub Docs.

🤖 Beep boop! This comment was added automatically by github/accessibility-alt-text-bot.

[github-actions]

size-limit report 📦

Path	Size
packages/react/dist/browser.esm.js	104.27 KB (0%)
packages/react/dist/browser.umd.js	104.59 KB (0%)

[joshblack]

Also @joshblack I think the SegmentedContorl has the same issue with leadingIcon where it does only accept a function but not an element. Can we fix that?

@lukasoppermann definitely! I think it should be just as doable as the last component with this issue 👍 This could be a good issue for FR to take on (adding support for element to loadingIcon) since it should be quick

[lukasoppermann]

After discussing with @siddharthkp we should keep the current behavior in the segmentedControl, as it otherwise leaves us open to unwanted customization in the icons.

I talked to Figma and they are working on a fix. We are hoping to get this in the next week, but until then this is blocked.

[github-actions]

👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks!

[langermank]

Good to go 🚀