(fix) adjust the Grommet button theme to prevent overwriting

AlexsLemonade · Sep 18, 2023 · 11b2e34 · 11b2e34
1 parent 602c350
commit 11b2e34
Showing 1 changed file with 7 additions and 11 deletions.
diff --git a/client/src/theme/button.js b/client/src/theme/button.js
@@ -1,28 +1,26 @@
+const defaultBorder = { radius: '4px', width: '1px' }
+
 export default {
-  border: {
-    radius: '4px',
-    width: '1px'
-  },
   color: 'brand',
   default: {
     color: 'brand',
-    border: { color: 'brand', width: '1px' },
+    border: { color: 'brand', ...defaultBorder },
     padding: {
       horizontal: '24px',
       vertical: '7px' // 8px - 1px for border
     }
   },
   primary: {
     background: { color: 'brand' },
-    border: { color: 'brand', width: '1px' },
+    border: { color: 'brand', ...defaultBorder },
     color: 'white',
     padding: {
       horizontal: '24px',
       vertical: '7px' // 8px - 1px for border
     }
   },
   secondary: {
-    border: { color: 'brand', width: '1px' },
+    border: { color: 'brand', ...defaultBorder },
     color: 'text',
     padding: {
       horizontal: '24px',
@@ -34,17 +32,15 @@ export default {
     color: 'text',
     secondary: {
       background: 'none',
-      border: {
-        color: 'brand-contrast'
-      }
+      border: { color: 'brand-contrast', ...defaultBorder }
     }
   },
   disabled: {
     opacity: 1,
     shadow: undefined,
     color: 'text-weak',
     background: 'black-tint-90',
-    border: { color: 'black-tint-90' },
+    border: { color: 'black-tint-90', ...defaultBorder },
     primary: { color: 'black-tint-40', background: 'black-tint-70' }
   },
   hover: {