feat(CC-batch-4): group review 5 updates

mattnolting · mattnolting · commit fe176efe7912 · 2025-08-13T15:54:23.000-04:00
diff --git a/packages/code-connect/components/DataList/_1BuildItYourselfBasicRows.figma.tsx b/packages/code-connect/components/DataList/_1BuildItYourselfBasicRows.figma.tsx
@@ -9,7 +9,7 @@ figma.connect(
       children: figma.children('*')
     },
     example: (props) => (
-      <DataListItem>
+      <DataListItem aria-labelledby="child-text-id">
         <DataListItemRow rowid="<row-id>">{props.children}</DataListItemRow>
       </DataListItem>
     )
diff --git a/packages/code-connect/components/DataList/_1BuildItYourselfClickableRows.figma.tsx b/packages/code-connect/components/DataList/_1BuildItYourselfClickableRows.figma.tsx
@@ -11,7 +11,7 @@ figma.connect(
       children: figma.children('*')
     },
     example: (props) => (
-      <DataListItem>
+      <DataListItem aria-labelledby="child text id">
         <DataListItemRow rowid="<row-id>">{props.children}</DataListItemRow>
       </DataListItem>
     )
diff --git a/packages/code-connect/components/DataList/_2BaseComponentsControlCell.figma.tsx b/packages/code-connect/components/DataList/_2BaseComponentsControlCell.figma.tsx
@@ -31,11 +31,11 @@ figma.connect(
   {
     props: {
       isExpandable: figma.boolean('Row expansion', {
-        true: <DataListToggle onClick={() => {}} isExpanded={false} id="m-ex-toggle1" aria-controls="m-ex-expand1" />,
+        true: <DataListToggle onClick={() => {}} isExpanded={false} id="toggle1" aria-controls="expand1" />,
         false: undefined
       }),
       isSelectable: figma.boolean('Row select', {
-        true: <DataListCheck id="ex1-item1" aria-labelledby="ex1-item1" name="ex1-item1" />,
+        true: <DataListCheck id="check-item1" aria-labelledby="item1" name="item1" />,
         false: undefined
       })
     },
diff --git a/packages/code-connect/components/DualListSelector/DualListHeader.figma.tsx b/packages/code-connect/components/DualListSelector/DualListHeader.figma.tsx
@@ -1,5 +1,5 @@
 import figma from '@figma/code-connect';
-import { Button, ButtonVariant, DualListSelectorPane } from '@patternfly/react-core';
+import { Button, ButtonVariant, DualListSelectorPane, SearchInput } from '@patternfly/react-core';
 import TimesIcon from '@patternfly/react-icons/dist/esm/icons/times-icon';
 
 // Documentation for DualListHeader can be found at https://www.patternfly.org/components/dual-list-selector
@@ -21,7 +21,9 @@ figma.connect(
         false: undefined
       }),
       isSearchable: figma.boolean('Has search bar', {
-        true: `buildSearchInput(true)`,
+        true: (
+          <SearchInput value={currentPaneFilters} onChange={() => {}} onClear={() => {}} aria-label={paneSearchLabel} />
+        ),
         false: undefined
       }),
       actions: figma.boolean('Has search bar', {
@@ -50,7 +52,7 @@ figma.connect(
     example: (props) => (
       // Documentation for DualListHeader can be found at https://www.patternfly.org/components/dual-list-selector
       <DualListSelectorPane
-        // actions={props.actions}
+        actions={props.actions}
         listMinHeight="300px"
         onSearch={props.onSearch}
         searchInput={props.isSearchable}
diff --git a/packages/code-connect/components/DualListSelector/DualListItemTree.figma.tsx b/packages/code-connect/components/DualListSelector/DualListItemTree.figma.tsx
@@ -8,7 +8,7 @@ figma.connect(
   'https://www.figma.com/design/aEBBvq0J3EPXxHvv6WgDx9/PatternFly-6--Components-Test?node-id=21279-116342',
   {
     props: {
-      options: `DualListSelectorTreeItemData[] = buildOptions(isChosen, data, false)`
+      options: 'see example for tree data'
     },
     example: (props) => (
       <DualListSelectorList>
diff --git a/packages/code-connect/components/DualListSelector/DualListItemTypes.figma.tsx b/packages/code-connect/components/DualListSelector/DualListItemTypes.figma.tsx

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ figma.connect(`
`9`	`9`	`children: figma.children('*')`
`10`	`10`	`},`
`11`	`11`	`example: (props) => (`
`12`		`- <DataListItem>`
	`12`	`+ <DataListItem aria-labelledby="child-text-id">`
`13`	`13`	`<DataListItemRow rowid="<row-id>">{props.children}</DataListItemRow>`
`14`	`14`	`</DataListItem>`
`15`	`15`	`)`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ figma.connect(`
`11`	`11`	`children: figma.children('*')`
`12`	`12`	`},`
`13`	`13`	`example: (props) => (`
`14`		`- <DataListItem>`
	`14`	`+ <DataListItem aria-labelledby="child text id">`
`15`	`15`	`<DataListItemRow rowid="<row-id>">{props.children}</DataListItemRow>`
`16`	`16`	`</DataListItem>`
`17`	`17`	`)`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ figma.connect(`
`8`	`8`	`'https://www.figma.com/design/aEBBvq0J3EPXxHvv6WgDx9/PatternFly-6--Components-Test?node-id=21279-116342',`
`9`	`9`	`{`
`10`	`10`	`props: {`
`11`		- options: `DualListSelectorTreeItemData[] = buildOptions(isChosen, data, false)`
	`11`	`+ options: 'see example for tree data'`
`12`	`12`	`},`
`13`	`13`	`example: (props) => (`
`14`	`14`	`<DualListSelectorList>`