ActionList: Ensure variant is respected when full

[TylerJDev]

When ActionList utilized the full variant, the style was not applying correctly (behind feature flag). This fix ensures that the variant is respected when the FF is enabled.

This change will only affect ActionList behind its feature flag (primer_react_action_list_item_as_button).

Changelog

Changed

• Adds conditional to ensure variant is respected when full and FF is enabled

Rollout strategy

• Patch release
• Minor release
• Major release; if selected, include a written rollout or migration plan
• None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

• Added/updated tests
• Added/updated documentation
• Added/updated previews (Storybook)
• Changes are SSR compatible
• Tested in Chrome
• Tested in Firefox
• Tested in Safari
• Tested in Edge
• (GitHub staff only) Integration tests pass at github/github (Learn more about how to run integration tests)

[changeset-bot]

🦋 Changeset detected

Latest commit: 1f96ef5

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

This PR includes changesets to release 1 package

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

[github-actions]

size-limit report 📦

Path	Size
packages/react/dist/browser.esm.js	91.11 KB (+0.16% 🔺)
packages/react/dist/browser.umd.js	91.29 KB (+0.06% 🔺)

[primer-integration]

👋 Hi from github/github! Your integration PR is ready: https://github.com/github/github/pull/331186

[joshblack]

Just wanted to ask if there was a good way to add a test for this kind of change. Is this something that VRT could help out with or would this be more of a Jest thing?

[TylerJDev]

Just wanted to ask if there was a good way to add a test for this kind of change. Is this something that VRT could help out with or would this be more of a Jest thing?

That's a good point! I think a VRT would be perfect! I believe this wasn't caught because we don't test for variant: full in our VRT e2e tests. I'll follow up with a story that utilizes variant: full.