Skip to content

Commit

Permalink
better handling of #15
Browse files Browse the repository at this point in the history
  • Loading branch information
yury-dymov committed Feb 4, 2018
1 parent ff968f2 commit 6bcba25
Showing 1 changed file with 15 additions and 5 deletions.
20 changes: 15 additions & 5 deletions src/AutoCompleteTextField.jsx
Expand Up @@ -39,10 +39,10 @@ const defaultProps = {
defaultValue: '',
disabled: false,
maxOptions: 6,
onBlur: () => {},
onChange: () => {},
onKeyDown: () => {},
onRequestOptions: () => {},
onBlur: () => { },
onChange: () => { },
onKeyDown: () => { },
onRequestOptions: () => { },
options: [],
regex: '^[A-Za-z0-9\\-_]+$',
matchAny: false,
Expand Down Expand Up @@ -259,7 +259,7 @@ class AutocompleteTextField extends React.Component {
handleSelection(idx) {
const { matchStart, matchLength, options } = this.state;

const slug = options[idx] || options[0];
const slug = options[idx];
const value = this.recentValue;
const part1 = value.substring(0, matchStart);
const part2 = value.substring(matchStart + matchLength);
Expand Down Expand Up @@ -323,6 +323,16 @@ class AutocompleteTextField extends React.Component {
const { maxOptions, offsetX, offsetY } = this.props;
const { value, left, matchStart, matchLength, options, selection, top } = this.state;

if (options.length === 0) {
return null;
}

if (selection >= options.length) {
this.setState({ selection: 0 });

return null;
}

const optionNumber = this.props.maxOptions === 0 ? options.length : maxOptions;

const helperOptions = options.slice(0, optionNumber).map((val, idx) => {
Expand Down

0 comments on commit 6bcba25

Please sign in to comment.