Removed redundant focus ring management

saas-js · Jun 7, 2022 · a8a51f7 · a8a51f7
1 parent 532a7d4
commit a8a51f7
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 17 deletions.
diff --git a/packages/saas-ui-theme/src/base/components/list.ts b/packages/saas-ui-theme/src/base/components/list.ts
@@ -23,9 +23,6 @@ const baseStyle: PartsStyleFunction<typeof parts> = (props) => {
       _focus: {
         boxShadow: 'outline',
         bg: mode('blackAlpha.200', 'whiteAlpha.300')(props),
-        '&:not(:focus-visible)': {
-          boxShadow: 'none',
-        },
       },
       _active: {
         bg: mode('blackAlpha.300', 'whiteAlpha.400')(props),

diff --git a/packages/saas-ui-theme/src/saas-ui/components/button.ts b/packages/saas-ui-theme/src/saas-ui/components/button.ts
@@ -188,13 +188,6 @@ export default {
   defaultProps: {
     size: 'sm',
   },
-  baseStyle: {
-    _focus: {
-      '&:not(:focus-visible)': {
-        boxShadow: 'none',
-      },
-    },
-  },
   variants: {
     solid: variantSolid,
     ghost: variantGhost,

diff --git a/packages/saas-ui-theme/src/saas-ui/components/link.ts b/packages/saas-ui-theme/src/saas-ui/components/link.ts
@@ -1,9 +1,3 @@
 export default {
-  baseStyle: {
-    _focus: {
-      '&:not(:focus-visible)': {
-        boxShadow: 'none',
-      },
-    },
-  },
+  baseStyle: {},
 }