Skip to content
This repository has been archived by the owner on Mar 1, 2024. It is now read-only.

Commit

Permalink
fix(Form): Fix invalid properties for maxLength, minLength, autoCompl…
Browse files Browse the repository at this point in the history
…ete (#440)

fix(Form): Fix invalid properties for maxLength, minLength, autoComplete
  • Loading branch information
jonthomp committed May 31, 2019
2 parents 57aced0 + d63f9c3 commit cb4a69a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
8 changes: 4 additions & 4 deletions src/components/Form/Form.react.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ export type Props = {|
+className?: string,
+action?: string,
+method?: string,
+autocomplete: "on" | "off",
+autoComplete: "on" | "off",
|};

function Form({
Expand All @@ -46,23 +46,23 @@ function Form({
action,
method,
onSubmit,
autocomplete,
autoComplete,
}: Props): React.Node {
return (
<form
className={className}
onSubmit={onSubmit}
action={action}
method={method}
autocomplete={autocomplete}
autoComplete={autoComplete}
>
{children}
</form>
);
}

Form.defaultProps = {
autocomplete: "off",
autoComplete: "off",
};

Form.Group = FormGroup;
Expand Down
12 changes: 6 additions & 6 deletions src/components/Form/FormInput.react.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ type FormStyle = {|
+value?: string | number,
+min?: string | number,
+max?: string | number,
+minlength?: string | number,
+maxlength?: string | number,
+minLength?: string | number,
+maxLength?: string | number,
+disabled?: boolean,
+readOnly?: boolean,
+autoFocus?: boolean,
Expand Down Expand Up @@ -74,8 +74,8 @@ function FormInput(props: Props): React.Node {
value,
min,
max,
minlength,
maxlength,
minLength,
maxLength,
checked,
onChange,
onMouseEnter,
Expand Down Expand Up @@ -120,8 +120,8 @@ function FormInput(props: Props): React.Node {
value,
min,
max,
minlength,
maxlength,
minLength,
maxLength,
disabled,
readOnly,
autoFocus,
Expand Down

0 comments on commit cb4a69a

Please sign in to comment.