fix(input): remove useless container

kodiak-packages · May 23, 2020 · 3fc4073 · 3fc4073
1 parent 93f3105
commit 3fc4073
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 22 deletions.
diff --git a/src/components/Input/Input.module.css b/src/components/Input/Input.module.css
@@ -1,7 +1,3 @@
-.container {
-  width: 100%;
-}
-
 .input {
   font: inherit;
   box-sizing: border-box;
@@ -10,6 +6,7 @@
   border-radius: var(--border-radius-small);
   border: 1px solid var(--color-neutral-4);
   padding: 6.5px 14px;
+  width: 100%;
 }
 
 .input::placeholder {

diff --git a/src/components/Input/Input.tsx b/src/components/Input/Input.tsx
@@ -36,27 +36,26 @@ const Input = React.forwardRef<HTMLInputElement, Props>(
     }: Props,
     ref,
   ) => {
-    const mergedContainerClassName = classNames(cssReset.ventura, styles.container, className);
-    const mergedInputClassName = classNames(styles.input, {
+    const inputClassNames = classNames(cssReset.ventura, styles.input, {
       [styles.containsError]: Boolean(isInvalid),
+      className,
     });
+
     return (
-      <div className={mergedContainerClassName} data-testid="container">
-        <input
-          name={name}
-          value={value}
-          onChange={onChange}
-          placeholder={placeholder}
-          type={type}
-          spellCheck={spellCheck}
-          autoComplete={autoComplete ? 'on' : 'off'}
-          maxLength={maxLength}
-          ref={ref}
-          className={mergedInputClassName}
-          data-testid={`input-${name}`}
-          onBlur={onBlur}
-        />
-      </div>
+      <input
+        name={name}
+        value={value}
+        onChange={onChange}
+        placeholder={placeholder}
+        type={type}
+        spellCheck={spellCheck}
+        autoComplete={autoComplete ? 'on' : 'off'}
+        maxLength={maxLength}
+        ref={ref}
+        className={inputClassNames}
+        data-testid={`input-${name}`}
+        onBlur={onBlur}
+      />
     );
   },
 );