fix(input): update right icon margin and padding

innovaccer · Jul 7, 2023 · 46e9fd8 · 46e9fd8
1 parent 4f34462
commit 46e9fd8
Show file tree

Hide file tree

Showing 8 changed files with 80 additions and 67 deletions.
diff --git a/core/components/atoms/input/Input.tsx b/core/components/atoms/input/Input.tsx
@@ -137,7 +137,7 @@ export interface InputProps extends BaseProps, BaseHtmlProps<HTMLInputElement> {
 const sizeMapping = {
   tiny: 12,
   regular: 16,
-  large: 20,
+  large: 16,
 };
 
 /**
@@ -192,6 +192,8 @@ export const Input = React.forwardRef<HTMLInputElement, InputProps>((props, forw
       [`Input--${size}`]: size,
       ['Input--disabled']: disabled || readOnly,
       ['Input--error']: error,
+      ['Input--leftIcon']: size !== 'tiny' && icon,
+      ['Input--rightIcon']: !disabled && (info || ((actionIcon || onClear) && (value || defaultValue))),
     },
     className
   );
@@ -211,6 +213,7 @@ export const Input = React.forwardRef<HTMLInputElement, InputProps>((props, forw
   const rightIconClass = classNames({
     ['Input-icon']: true,
     ['Input-iconWrapper--right']: true,
+    ['ml-4']: true,
   });
 
   const trigger = (

diff --git a/core/components/atoms/input/__tests__/__snapshots__/Input.test.tsx.snap b/core/components/atoms/input/__tests__/__snapshots__/Input.test.tsx.snap
@@ -30,7 +30,7 @@ exports[`Input component
 <body>
   <div>
     <div
-      class="Input Input--regular"
+      class="Input Input--regular Input--leftIcon"
       data-test="DesignSystem-InputWrapper"
       role="presentation"
       style="min-width: 256px;"
@@ -89,7 +89,7 @@ exports[`Input component
 <body>
   <div>
     <div
-      class="Input Input--regular"
+      class="Input Input--regular Input--leftIcon"
       data-test="DesignSystem-InputWrapper"
       role="presentation"
       style="min-width: 256px;"
@@ -382,7 +382,7 @@ exports[`Input component
 <body>
   <div>
     <div
-      class="Input Input--regular Input--disabled"
+      class="Input Input--regular Input--disabled Input--leftIcon"
       data-test="DesignSystem-InputWrapper"
       role="presentation"
       style="min-width: 256px;"
@@ -443,7 +443,7 @@ exports[`Input component
 <body>
   <div>
     <div
-      class="Input Input--regular Input--disabled"
+      class="Input Input--regular Input--disabled Input--leftIcon"
       data-test="DesignSystem-InputWrapper"
       role="presentation"
       style="min-width: 256px;"
@@ -502,7 +502,7 @@ exports[`Input component
 <body>
   <div>
     <div
-      class="Input Input--regular Input--error"
+      class="Input Input--regular Input--error Input--leftIcon"
       data-test="DesignSystem-InputWrapper"
       role="presentation"
       style="min-width: 256px;"
@@ -561,7 +561,7 @@ exports[`Input component
 <body>
   <div>
     <div
-      class="Input Input--regular Input--disabled"
+      class="Input Input--regular Input--disabled Input--leftIcon"
       data-test="DesignSystem-InputWrapper"
       role="presentation"
       style="min-width: 256px;"

diff --git a/core/components/atoms/input/actionButton/ActionButton.tsx b/core/components/atoms/input/actionButton/ActionButton.tsx
@@ -48,6 +48,7 @@ export const ActionButton = (props: ActionButtonProps) => {
     [`material-icons-${type}`]: type && type !== 'filled',
     ['ActionButton']: true,
     [`${className}`]: className,
+    ['ml-4']: true,
   });
 
   const styles = {

diff --git a/core/components/atoms/input/actionButton/__tests__/__snapshots__/ActionButton.test.tsx.snap b/core/components/atoms/input/actionButton/__tests__/__snapshots__/ActionButton.test.tsx.snap
@@ -6,7 +6,7 @@ exports[`ActionButton component
 <body>
   <div>
     <i
-      class="material-icons material-icons-round ActionButton"
+      class="material-icons material-icons-round ActionButton ml-4"
       data-test="DesignSystem-Input-ActionButton"
       role="button"
       style="font-size: 16px; width: 16px;"
@@ -24,7 +24,7 @@ exports[`ActionButton component
 <body>
   <div>
     <i
-      class="material-icons ActionButton"
+      class="material-icons ActionButton ml-4"
       data-test="DesignSystem-Input-ActionButton"
       role="button"
       style="font-size: 16px; width: 16px;"
@@ -42,7 +42,7 @@ exports[`ActionButton component
 <body>
   <div>
     <i
-      class="material-icons material-icons-outlined ActionButton"
+      class="material-icons material-icons-outlined ActionButton ml-4"
       data-test="DesignSystem-Input-ActionButton"
       role="button"
       style="font-size: 16px; width: 16px;"
@@ -60,7 +60,7 @@ exports[`ActionButton component
 <body>
   <div>
     <i
-      class="material-icons material-icons-round ActionButton"
+      class="material-icons material-icons-round ActionButton ml-4"
       data-test="DesignSystem-Input-ActionButton"
       role="button"
       style="font-size: 16px; width: 16px;"
@@ -78,7 +78,7 @@ exports[`ActionButton component
 <body>
   <div>
     <i
-      class="material-icons material-icons-sharp ActionButton"
+      class="material-icons material-icons-sharp ActionButton ml-4"
       data-test="DesignSystem-Input-ActionButton"
       role="button"
       style="font-size: 16px; width: 16px;"
@@ -96,7 +96,7 @@ exports[`ActionButton component
 <body>
   <div>
     <i
-      class="material-icons material-icons-two-tone ActionButton"
+      class="material-icons material-icons-two-tone ActionButton ml-4"
       data-test="DesignSystem-Input-ActionButton"
       role="button"
       style="font-size: 16px; width: 16px;"

diff --git a/core/components/organisms/datePicker/__tests__/__snapshots__/DatePicker.test.tsx.snap b/core/components/organisms/datePicker/__tests__/__snapshots__/DatePicker.test.tsx.snap
@@ -5157,7 +5157,7 @@ exports[`DatePicker component snapshots renders snapshot for prop:withInput
         data-test="DesignSystem-InputMask--Wrapper"
       >
         <div
-          class="Input Input--regular"
+          class="Input Input--regular Input--leftIcon Input--rightIcon"
           data-test="DesignSystem-InputWrapper"
           role="presentation"
           style="min-width: 256px;"
@@ -5182,7 +5182,7 @@ exports[`DatePicker component snapshots renders snapshot for prop:withInput
             value="03/01/2020"
           />
           <div
-            class="Input-icon Input-iconWrapper--right"
+            class="Input-icon Input-iconWrapper--right ml-4"
           >
             <i
               class="material-icons material-icons-round Icon Input-icon--right"