Merge pull request #30192 from s-alves10/fix/issue-26985

fix: vertical centering button text without padding
Expensify · Oct 25, 2023 · 1e22431 · 1e22431
2 parents f19970f + 3271eb0
commit 1e22431
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/styles/styles.ts b/src/styles/styles.ts
@@ -480,7 +480,9 @@ const styles = (theme: ThemeDefault) =>
             borderRadius: variables.buttonBorderRadius,
             minHeight: variables.componentSizeLarge,
             justifyContent: 'center',
+            alignItems: 'center',
             ...spacing.ph3,
+            ...spacing.pv0,
         },
 
         buttonContainer: {
@@ -507,28 +509,22 @@ const styles = (theme: ThemeDefault) =>
         buttonSmall: {
             borderRadius: variables.buttonBorderRadius,
             minHeight: variables.componentSizeSmall,
-            paddingTop: 4,
             paddingHorizontal: 14,
-            paddingBottom: 4,
             backgroundColor: theme.buttonDefaultBG,
         },
 
         buttonMedium: {
             borderRadius: variables.buttonBorderRadius,
             minHeight: variables.componentSizeNormal,
-            paddingTop: 12,
             paddingRight: 16,
-            paddingBottom: 12,
             paddingLeft: 16,
             backgroundColor: theme.buttonDefaultBG,
         },
 
         buttonLarge: {
             borderRadius: variables.buttonBorderRadius,
             minHeight: variables.componentSizeLarge,
-            paddingTop: 8,
             paddingRight: 10,
-            paddingBottom: 8,
             paddingLeft: 10,
             backgroundColor: theme.buttonDefaultBG,
         },
@@ -609,7 +605,6 @@ const styles = (theme: ThemeDefault) =>
         },
 
         buttonCTA: {
-            paddingVertical: 6,
             ...spacing.mh4,
         },