Allow custom children in ActionItem

[dgreif]

Some use cases for ActionList have revealed that the built-in rendering is not always providing enough flexibility, such as if you want to render the text inside a <Label>. Although we can override rendering with renderItem, this fully replaces the entire item and removes helpful tools like onAction and the selected checkmark. Additionally, the requirement of text on ItemProps forces consumers to convert existing data structures into a secondary data structure in order to provide text. This change makes all fields optional (which opens it up for use by any item interface), and adds a children prop that will render a custom ReactNode before the text section of the item.

Screenshots

Merge checklist

• Added/updated tests
• Added/updated documentation
• Tested in Chrome
• Tested in Firefox
• Tested in Safari
• Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/primer/primer-components/CvhfL166KaFEGLfBxapmui6xYh6y
✅ Preview: https://primer-components-git-action-item-children-primer.vercel.app

[changeset-bot]

🦋 Changeset detected

Latest commit: 0468584

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

This PR includes changesets to release 1 package

Name	Type
@primer/components	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

[VanAnderson]

Looks pretty straight forward - non blocking but maybe we could update the docs to at least list the new option to use children?

[dgreif]

@VanAnderson I couldn't find any docs that explicitly define ItemProps. Happy to update them if I just overlooked them. For now, I think it's just examples within the ActionList docs. This is an area that we will need to revisit when we have solidified the API I think.

[smockle]

LGTM! 😎 Nice work on this!