Fix ActionList.Item conditional when FF is not enabled (#4695)

* Fix conditional when FF is not enabled

* Add changeset

Author: TylerJDev

.changeset/modern-cooks-decide.md

@@ -0,0 +1,5 @@
+---
+'@primer/react': patch
+---
+
+Fixes conditional in `ActionList.Item` that was dependent on FF being active before applying forwarded ref.

packages/react/src/ActionList/ActionList.test.tsx

@@ -426,6 +426,34 @@ describe('ActionList', () => {
     expect(listItems.length).toBe(2)
   })
 
+  it('should apply ref to ActionList.Item when feature flag is disabled', async () => {
+    const MockComponent = () => {
+      const ref = React.useRef<HTMLLIElement>(null)
+
+      const focusRef = () => {
+        if (ref.current) ref.current.focus()
+      }
+
+      return (
+        <FeatureFlags flags={{primer_react_action_list_item_as_button: false}}>
+          <button onClick={focusRef}>Prompt</button>
+          <ActionList>
+            <ActionList.Item ref={ref}>Item 1</ActionList.Item>
+            <ActionList.Item>Item 2</ActionList.Item>
+          </ActionList>
+        </FeatureFlags>
+      )
+    }
+
+    const {getByRole} = HTMLRender(<MockComponent />)
+    const triggerBtn = getByRole('button', {name: 'Prompt'})
+    const focusTarget = getByRole('listitem', {name: 'Item 1'})
+
+    fireEvent.click(triggerBtn)
+
+    expect(document.activeElement).toBe(focusTarget)
+  })
+
   it('should render ActionList.Item as li when feature flag is enabled and has proper aria role', async () => {
     const {container} = HTMLRender(
       <FeatureFlags flags={{primer_react_action_list_item_as_button: false}}>

packages/react/src/ActionList/Item.tsx

@@ -368,7 +368,7 @@ export const Item = React.forwardRef<HTMLLIElement, ActionListItemProps>(
         value={{variant, disabled, inactive: Boolean(inactiveText), inlineDescriptionId, blockDescriptionId}}
       >
         <LiBox
-          ref={buttonSemanticsFeatureFlag || listSemantics ? forwardedRef : null}
+          ref={!buttonSemanticsFeatureFlag || listSemantics ? forwardedRef : null}
           sx={
             buttonSemanticsFeatureFlag
               ? merge<BetterSystemStyleObject>(