feat(CC-batch-5): review-athon round 1 followup

Author: mattnolting

packages/code-connect/components/EmptyState/EmptyState.figma.tsx

@@ -2,16 +2,15 @@ import figma from '@figma/code-connect';
 import { EmptyState, EmptyStateBody, EmptyStateFooter, EmptyStateActions } from '@patternfly/react-core';
 import PlusCircleIcon from '@patternfly/react-icons/dist/esm/icons/plus-circle-icon';
 
-// TODO: FIGMA: Add Empty state footer
-// TODO: FIGMA: Consolodate empty state examples
-// TODO: FIGMA: Add empty state footer
-// TODO: FIGMA: Add empty state actions
-// TODO: FIGMA: Add empty state icon
-// TODO: FIGMA: Add empty state title
-// TODO: FIGMA: Add empty state body
-// TODO: FIGMA: Add empty state variant
-// TODO: FIGMA: Add empty state status
-// TODO: FIGMA: Add empty state isLoading
+// TODO: DESIGN: Add Empty state footer
+// TODO: DESIGN: Consolodate empty state examples
+// TODO: DESIGN: Add empty state footer
+// TODO: DESIGN: Add empty state actions
+// TODO: DESIGN: Add empty state icon
+// TODO: DESIGN: Add empty state title
+// TODO: DESIGN: Add empty state body
+// TODO: DESIGN: Add empty state variant
+// TODO: DESIGN: Add empty state status
 // Based on Code Connect's limitations, this component needs to be overhauled. Using the base component approach present in
 // other components would significantly reduce complexity.
 
@@ -44,6 +43,7 @@ figma.connect(
     },
     example: (props) => (
       <EmptyState
+        // this may need to be updated for accessibility
         headingLevel="h4"
         icon={<PlusCircleIcon />}
         variant={props.variant}

packages/code-connect/components/ExpandableSection/ExandableSectionTruncate.figma.tsx

@@ -1,46 +1,31 @@
 import figma from '@figma/code-connect';
 import { ExpandableSection } from '@patternfly/react-core';
 
-// TODO: FIGMA: Create toggle component
+// TODO: DESIGN: Create toggle component
 // onToggle={() => {}} // only required if the user wants to be able to have other side effects when opening/closing
 // isExpanded={() => {}} // only required if the user wants to be able to have other side effects when opening/closing
 
 // Documentation for ExpandableSection can be found at https://www.patternfly.org/components/expandable-section
 
-const toggleContent = `
-    <div>
-    <span>You can also use icons </span>
-    <CheckCircleIcon />
-    <span> or badges </span>
-    <Badge isRead={true}>4</Badge>
-    <span> !</span>
-  </div>
-`;
-
 figma.connect(
   ExpandableSection,
   'https://www.figma.com/design/aEBBvq0J3EPXxHvv6WgDx9/PatternFly-6--Components-Test?node-id=2769-146',
   {
     props: {
       // enum
       isIndented: figma.enum('State', { 'Expanded Indent': true }),
-      toggleContent: figma.enum('State', { Expanded: toggleContent }),
       toggleTextCollapsed: figma.enum('State', { Collapsed: figma.string('Toggle Text More') }),
       toggleTextExpanded: figma.enum('State', { Expanded: figma.string('Toggle Text Less') }),
 
       // boolean
       expandedContentSectionText: figma.enum('State', {
         Default: figma.string('Default Truncate Text'),
         Expanded: figma.string('Expanded Truncate Text')
-      }),
-      isExpanded: figma.enum('State', { Expanded: true }),
-
-      children: figma.children('*')
+      })
     },
     example: (props) => (
       <ExpandableSection
         isIndented={props.isIndented}
-        toggleContent={props.toggleContent}
         toggleTextCollapsed={props.toggleTextCollapsed}
         toggleTextExpanded={props.toggleTextExpanded}
         variant="truncate"

packages/code-connect/components/ExpandableSection/ExpandableSection.figma.tsx

@@ -1,9 +1,9 @@
 import figma from '@figma/code-connect';
 import { ExpandableSection } from '@patternfly/react-core';
 
-// TODO: FIGMA: This component needs to be overhauled. Using the base component approach present in
+// TODO: DESIGN: This component needs to be overhauled. Using the base component approach present in
 // other components would significantly reduce complexity.
-// TODO: FIGMA: Create toggle component
+// TODO: DESIGN: Create toggle component
 // onToggle={() => {}} // only required if the user wants to be able to have other side effects when opening/closing
 // isExpanded={() => {}} // only required if the user wants to be able to have other side effects when opening/closing
 

packages/code-connect/components/ExpandableSection/ExpandableSectionDisclosure.figma.tsx

@@ -18,16 +18,14 @@ figma.connect(
         'Expanded Indent': true,
         'Expanded Custom Content': true
       }),
-      isIndented: figma.enum('State', { 'Expanded Indent': true }),
-
-      children: figma.children('*')
+      isIndented: figma.enum('State', { 'Expanded Indent': true })
     },
     example: (props) => (
       // Documentation for ExpandableSection can be found at https://www.patternfly.org/components/expandable-section
       <ExpandableSection
         isExpanded={props.isExpanded}
         isIndented={props.isIndented}
-        toggleContent={props.children}
+        onToggle={() => {}}
         toggleTextCollapsed={props.toggleTextCollapsed}
         toggleTextExpanded={props.toggleTextExpanded}
         variant="truncate"

packages/code-connect/components/FileUpload/MultipleFileUpload.figma.tsx

@@ -8,12 +8,12 @@ import {
 } from '@patternfly/react-core';
 import UploadIcon from '@patternfly/react-icons/dist/esm/icons/upload-icon';
 
-// TODO: FIGMA: Add status toggle text
-// TODO: FIGMA: Add status toggle icon
-// TODO: FIGMA: Add text separator
-// TODO: FIGMA: Add info text
-// TODO: FIGMA: Add status toggle text
-// TODO: FIGMA: Add status toggle icon
+// TODO: DESIGN: Add status toggle text
+// TODO: DESIGN: Add status toggle icon
+// TODO: DESIGN: Add text separator
+// TODO: DESIGN: Add info text
+// TODO: DESIGN: Add status toggle text
+// TODO: DESIGN: Add status toggle icon
 
 figma.connect(
   FileUpload,

packages/code-connect/components/FileUpload/SimpleFileUpload.figma.tsx

@@ -1,9 +1,9 @@
 import figma from '@figma/code-connect';
 import { FileUpload, FileUploadHelperText, HelperText, HelperTextItem } from '@patternfly/react-core';
 
-// TODO: FIGMA: Add filename placeholder
-// TODO: FIGMA: Add browse button text
-// TODO: FIGMA: Add FileUploadHelperText
+// TODO: DESIGN: Add filename placeholder
+// TODO: DESIGN: Add browse button text
+// TODO: DESIGN: Add FileUploadHelperText
 
 figma.connect(
   FileUpload,

packages/code-connect/components/Hint/Hint.figma.tsx

@@ -12,10 +12,10 @@ import {
 import EllipsisVIcon from '@patternfly/react-icons/dist/esm/icons/ellipsis-v-icon';
 import MenuToggle from '@patternfly/react-core/dist/js/components/MenuToggle/MenuToggle';
 
-// TODO: FIGMA: Use dropdown from figma for actions
-// TODO: FIGMA: Create hint title
-// TODO: FIGMA: Create hint body
-// TODO: FIGMA: Create hint footer
+// TODO: DESIGN: Use dropdown from figma for actions
+// TODO: DESIGN: Create hint title
+// TODO: DESIGN: Create hint body
+// TODO: DESIGN: Create hint footer
 
 // Note: Figma will not render conditional props within React components.
 // This means that the <HintTitle>, <HintBody>, and <HintFooter> tags will not be properly render figma.string()