Revert padding on ActionList.

primer · Feb 28, 2023 · 235e4cf · 235e4cf
1 parent ae0cbde
commit 235e4cf
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/src/ActionList/List.tsx b/src/ActionList/List.tsx
@@ -53,7 +53,7 @@ export const List = React.forwardRef<HTMLUListElement, ActionListProps>(
     const styles = {
       margin: 0,
       paddingInlineStart: 0, // reset ul styles
-      paddingY: 0,
+      paddingY: variant === 'inset' ? 2 : 0,
     }
 
     /** if list is inside a Menu, it will get a role from the Menu */

diff --git a/src/NavList/__snapshots__/NavList.test.tsx.snap b/src/NavList/__snapshots__/NavList.test.tsx.snap
@@ -26,8 +26,8 @@ exports[`NavList renders a simple list 1`] = `
 .c0 {
   margin: 0;
   padding-inline-start: 0;
-  padding-top: 0;
-  padding-bottom: 0;
+  padding-top: 8px;
+  padding-bottom: 8px;
 }
 
 .c2 {
@@ -429,8 +429,8 @@ exports[`NavList renders with groups 1`] = `
 .c0 {
   margin: 0;
   padding-inline-start: 0;
-  padding-top: 0;
-  padding-bottom: 0;
+  padding-top: 8px;
+  padding-bottom: 8px;
 }
 
 .c6 {
@@ -876,8 +876,8 @@ exports[`NavList.Item with NavList.SubNav does not have active styles if SubNav
 .c0 {
   margin: 0;
   padding-inline-start: 0;
-  padding-top: 0;
-  padding-bottom: 0;
+  padding-top: 8px;
+  padding-bottom: 8px;
 }
 
 .c10 {
@@ -1352,8 +1352,8 @@ exports[`NavList.Item with NavList.SubNav has active styles if SubNav contains t
 .c0 {
   margin: 0;
   padding-inline-start: 0;
-  padding-top: 0;
-  padding-bottom: 0;
+  padding-top: 8px;
+  padding-bottom: 8px;
 }
 
 .c10 {

diff --git a/src/__tests__/__snapshots__/ActionList.test.tsx.snap b/src/__tests__/__snapshots__/ActionList.test.tsx.snap
@@ -4,8 +4,8 @@ exports[`ActionList renders consistently 1`] = `
 .c0 {
   margin: 0;
   padding-inline-start: 0;
-  padding-top: 0;
-  padding-bottom: 0;
+  padding-top: 8px;
+  padding-bottom: 8px;
 }
 
 <ul