mirror of https://github.com/vector-im/riot-web
Add focus handling to validation
Update the Field component and validation handling to show / hide validation feedback on focus / blur events.pull/21833/head
parent
338d83ab55
commit
87f13cfe55
|
@ -53,20 +53,53 @@ export default class Field extends React.PureComponent {
|
||||||
};
|
};
|
||||||
}
|
}
|
||||||
|
|
||||||
onChange = (ev) => {
|
onFocus = (ev) => {
|
||||||
if (this.props.onValidate) {
|
this.validate({
|
||||||
const result = this.props.onValidate(ev.target.value);
|
value: ev.target.value,
|
||||||
this.setState({
|
focused: true,
|
||||||
valid: result.valid,
|
|
||||||
feedback: result.feedback,
|
|
||||||
});
|
});
|
||||||
|
// Parent component may have supplied its own `onFocus` as well
|
||||||
|
if (this.props.onFocus) {
|
||||||
|
this.props.onFocus(ev);
|
||||||
}
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
onChange = (ev) => {
|
||||||
|
this.validate({
|
||||||
|
value: ev.target.value,
|
||||||
|
focused: true,
|
||||||
|
});
|
||||||
// Parent component may have supplied its own `onChange` as well
|
// Parent component may have supplied its own `onChange` as well
|
||||||
if (this.props.onChange) {
|
if (this.props.onChange) {
|
||||||
this.props.onChange(ev);
|
this.props.onChange(ev);
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
onBlur = (ev) => {
|
||||||
|
this.validate({
|
||||||
|
value: ev.target.value,
|
||||||
|
focused: false,
|
||||||
|
});
|
||||||
|
// Parent component may have supplied its own `onBlur` as well
|
||||||
|
if (this.props.onBlur) {
|
||||||
|
this.props.onBlur(ev);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
validate({ value, focused }) {
|
||||||
|
if (!this.props.onValidate) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
const { valid, feedback } = this.props.onValidate({
|
||||||
|
value,
|
||||||
|
focused,
|
||||||
|
});
|
||||||
|
this.setState({
|
||||||
|
valid,
|
||||||
|
feedback,
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
render() {
|
render() {
|
||||||
const { element, prefix, onValidate, children, ...inputProps } = this.props;
|
const { element, prefix, onValidate, children, ...inputProps } = this.props;
|
||||||
|
|
||||||
|
@ -76,7 +109,9 @@ export default class Field extends React.PureComponent {
|
||||||
inputProps.type = inputProps.type || "text";
|
inputProps.type = inputProps.type || "text";
|
||||||
inputProps.placeholder = inputProps.placeholder || inputProps.label;
|
inputProps.placeholder = inputProps.placeholder || inputProps.label;
|
||||||
|
|
||||||
|
inputProps.onFocus = this.onFocus;
|
||||||
inputProps.onChange = this.onChange;
|
inputProps.onChange = this.onChange;
|
||||||
|
inputProps.onBlur = this.onBlur;
|
||||||
|
|
||||||
const fieldInput = React.createElement(inputElement, inputProps, children);
|
const fieldInput = React.createElement(inputElement, inputProps, children);
|
||||||
|
|
||||||
|
|
|
@ -34,8 +34,7 @@ import classNames from 'classnames';
|
||||||
* the overall validity and a feedback UI that can be rendered for more detail.
|
* the overall validity and a feedback UI that can be rendered for more detail.
|
||||||
*/
|
*/
|
||||||
export default function withValidation({ description, rules }) {
|
export default function withValidation({ description, rules }) {
|
||||||
return function onValidate(value) {
|
return function onValidate({ value, focused }) {
|
||||||
// TODO: Hide on blur
|
|
||||||
// TODO: Re-run only after ~200ms of inactivity
|
// TODO: Re-run only after ~200ms of inactivity
|
||||||
if (!value) {
|
if (!value) {
|
||||||
return {
|
return {
|
||||||
|
@ -73,6 +72,14 @@ export default function withValidation({ description, rules }) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Hide feedback when not focused
|
||||||
|
if (!focused) {
|
||||||
|
return {
|
||||||
|
valid,
|
||||||
|
feedback: null,
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
let details;
|
let details;
|
||||||
if (results && results.length) {
|
if (results && results.length) {
|
||||||
details = <ul className="mx_Validation_details">
|
details = <ul className="mx_Validation_details">
|
||||||
|
|
Loading…
Reference in New Issue