Add back check for GroupContext in ActionList.Selection to fix issues…

… introduced to github/github (#3267) * Add back check for groupContext in ActionList Selection component. * Update src/ActionList/Selection.tsx Co-authored-by: Josh Black <[email protected]> * chore: run format * Linting. --------- Co-authored-by: Josh Black <[email protected]> Co-authored-by: Josh Black <[email protected]>
primer · Jun 21, 2023 · 9a5c23c · 9a5c23c
1 parent b668d74
commit 9a5c23c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/ActionList/Selection.tsx b/src/ActionList/Selection.tsx
@@ -1,16 +1,18 @@
 import React from 'react'
 import {CheckIcon} from '@primer/octicons-react'
-import {ListContext, ActionListProps} from './List'
+import {ListContext} from './List'
 import {ActionListItemProps} from './shared'
 import {LeadingVisualContainer} from './Visuals'
+import {GroupContext} from './Group'
 
 type SelectionProps = Pick<ActionListItemProps, 'selected'>
 export const Selection: React.FC<React.PropsWithChildren<SelectionProps>> = ({selected}) => {
+  const {selectionVariant: groupSelectionVariant} = React.useContext(GroupContext)
   const {selectionVariant: listSelectionVariant} = React.useContext(ListContext)
 
   /** selectionVariant in Group can override the selectionVariant in List root */
   /** fallback to selectionVariant from container menu if any (ActionMenu, SelectPanel ) */
-  const selectionVariant: ActionListProps['selectionVariant'] = listSelectionVariant
+  const selectionVariant = groupSelectionVariant ?? listSelectionVariant
 
   if (!selectionVariant) {
     // if selectionVariant is not set on List, but Item is selected