TEST: Revert "feat(ActionList sub components) Convert to CSS Modules"

e2e/components/ActionList.test.ts

@@ -313,38 +313,33 @@ test.describe('ActionList', () => {
     }
   })
 
-  // removing this temporarily as there is a slight diff betqeen default and enabled CSS feature flag that feels like a non-issue
-  // eslint-disable-next-line jest/no-commented-out-tests
-  // test.describe('Text Wrap And Truncation', () => {
-  //   for (const theme of themes) {
-  // eslint-disable-next-line jest/no-commented-out-tests
-  //     test.describe(theme, () => {
-  // eslint-disable-next-line jest/no-commented-out-tests
-  //       test('default @vrt', async ({page}) => {
-  //         await visit(page, {
-  //           id: 'components-actionlist-features--text-wrap-and-truncation',
-  //           globals: {
-  //             colorScheme: theme,
-  //           },
-  //         })
-
-  //         // Default state
-  //         expect(await page.screenshot()).toMatchSnapshot(`ActionList.Text Wrap And Truncation.${theme}.png`)
-  //       })
-
-  // eslint-disable-next-line jest/no-commented-out-tests
-  //       test('axe @aat', async ({page}) => {
-  //         await visit(page, {
-  //           id: 'components-actionlist-features--text-wrap-and-truncation',
-  //           globals: {
-  //             colorScheme: theme,
-  //           },
-  //         })
-  //         await expect(page).toHaveNoViolations()
-  //       })
-  //     })
-  //   }
-  // })
+  test.describe('Text Wrap And Truncation', () => {
+    for (const theme of themes) {
+      test.describe(theme, () => {
+        test('default @vrt', async ({page}) => {
+          await visit(page, {
+            id: 'components-actionlist-features--text-wrap-and-truncation',
+            globals: {
+              colorScheme: theme,
+            },
+          })
+
+          // Default state
+          expect(await page.screenshot()).toMatchSnapshot(`ActionList.Text Wrap And Truncation.${theme}.png`)
+        })
+
+        test('axe @aat', async ({page}) => {
+          await visit(page, {
+            id: 'components-actionlist-features--text-wrap-and-truncation',
+            globals: {
+              colorScheme: theme,
+            },
+          })
+          await expect(page).toHaveNoViolations()
+        })
+      })
+    }
+  })
 
   test.describe('With Avatars', () => {
     for (const theme of themes) {
@@ -745,60 +740,4 @@ test.describe('ActionList', () => {
       })
     }
   })
-
-  test.describe('Visuals with Classnames', () => {
-    for (const theme of themes) {
-      test.describe(theme, () => {
-        test('default @vrt', async ({page}) => {
-          await visit(page, {
-            id: 'components-actionlist-dev--visual-custom-classname',
-            globals: {
-              colorScheme: theme,
-            },
-          })
-
-          // Default state
-          expect(await page.screenshot()).toMatchSnapshot(`Visuals with Classnames.${theme}.png`)
-        })
-
-        test('axe @aat', async ({page}) => {
-          await visit(page, {
-            id: 'components-actionlist-dev--visual-custom-classname',
-            globals: {
-              colorScheme: theme,
-            },
-          })
-          await expect(page).toHaveNoViolations()
-        })
-      })
-    }
-  })
-
-  test.describe('Link Item Options', () => {
-    for (const theme of themes) {
-      test.describe(theme, () => {
-        test('default @vrt', async ({page}) => {
-          await visit(page, {
-            id: 'components-actionlist-examples--list-link-item',
-            globals: {
-              colorScheme: theme,
-            },
-          })
-
-          // Default state
-          expect(await page.screenshot()).toMatchSnapshot(`Link Item Options.${theme}.png`)
-        })
-
-        test('axe @aat', async ({page}) => {
-          await visit(page, {
-            id: 'components-actionlist-examples--list-link-item',
-            globals: {
-              colorScheme: theme,
-            },
-          })
-          await expect(page).toHaveNoViolations()
-        })
-      })
-    }
-  })
 })

packages/react/src/ActionList/ActionList.dev.stories.tsx

@@ -7,7 +7,6 @@ import {Group} from './Group'
 import {Divider} from './Divider'
 import {Description} from './Description'
 import Avatar from '../Avatar'
-import {FileDirectoryIcon, HeartFillIcon} from '@primer/octicons-react'
 
 export default {
   title: 'Components/ActionList/Dev',
@@ -145,26 +144,3 @@ export const HeadingCustomClassname = () => (
     </ActionList.Group>
   </ActionList>
 )
-
-export const DescriptionCustomClassname = () => (
-  <ActionList>
-    <ActionList.Item>
-      Label
-      <ActionList.Description className="testCustomClassnameColor">This is a description</ActionList.Description>
-    </ActionList.Item>
-  </ActionList>
-)
-
-export const VisualCustomClassname = () => (
-  <ActionList>
-    <ActionList.Item>
-      Label
-      <ActionList.LeadingVisual className="testCustomClassnameColor">
-        <FileDirectoryIcon />
-      </ActionList.LeadingVisual>
-      <ActionList.TrailingVisual className="testCustomClassnameColor">
-        <HeartFillIcon />
-      </ActionList.TrailingVisual>
-    </ActionList.Item>
-  </ActionList>
-)

packages/react/src/ActionList/ActionList.examples.stories.tsx

@@ -94,21 +94,9 @@ export const ListLinkItem = () => (
       <ActionList.LeadingVisual>
         <LinkIcon />
       </ActionList.LeadingVisual>
-      With inline description
+      ActionList.LinkItem with everything
       <ActionList.Description variant="inline">inline description</ActionList.Description>
-    </ActionList.LinkItem>
-    <ActionList.LinkItem href="?path=/story/components-actionlist--default">
-      <ActionList.LeadingVisual>
-        <LinkIcon />
-      </ActionList.LeadingVisual>
-      With block description
       <ActionList.Description variant="block">Block description</ActionList.Description>
-    </ActionList.LinkItem>
-    <ActionList.LinkItem href="?path=/story/components-actionlist--default">
-      <ActionList.LeadingVisual>
-        <LinkIcon />
-      </ActionList.LeadingVisual>
-      Trailing visual
       <ActionList.TrailingVisual>⌘ + L</ActionList.TrailingVisual>
     </ActionList.LinkItem>
   </ActionList>