Skip to content

Commit

Permalink
Merge pull request #291 from PRGfx/task/selected-option-index-in-status
Browse files Browse the repository at this point in the history
Pass selected option's index to Status; use in tStatusSelectedOption
  • Loading branch information
hannalaakso committed Aug 9, 2018
2 parents b5ba980 + 1a7d0b0 commit 2ae9940
Show file tree
Hide file tree
Showing 9 changed files with 11 additions and 9 deletions.
2 changes: 1 addition & 1 deletion README.md
Expand Up @@ -291,7 +291,7 @@ Type: `Function`

A function that receives no arguments and should return the text that is used in the accessibility hint to indicate that there are no results.

#### `tStatusSelectedOption` (default: `` (selectedOption, length) => `${selectedOption} (1 of ${length}) is selected.` ``)
#### `tStatusSelectedOption` (default: `` (selectedOption, length, index) => `${selectedOption} (${index + 1} of ${length}) is selected.` ``)

Type: `Function`

Expand Down
2 changes: 1 addition & 1 deletion dist/accessible-autocomplete.min.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/accessible-autocomplete.min.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/lib/accessible-autocomplete.preact.min.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/lib/accessible-autocomplete.preact.min.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/lib/accessible-autocomplete.react.min.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/lib/accessible-autocomplete.react.min.js.map

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions src/autocomplete.js
Expand Up @@ -453,6 +453,7 @@ export default class Autocomplete extends Component {
queryLength={query.length}
minQueryLength={minLength}
selectedOption={this.templateInputValue(options[selected])}
selectedOptionIndex={selected}
tQueryTooShort={tStatusQueryTooShort}
tNoResults={tStatusNoResults}
tSelectedOption={tStatusSelectedOption}
Expand Down
5 changes: 3 additions & 2 deletions src/status.js
Expand Up @@ -4,7 +4,7 @@ export default class Status extends Component {
static defaultProps = {
tQueryTooShort: (minQueryLength) => `Type in ${minQueryLength} or more characters for results.`,
tNoResults: () => 'No search results.',
tSelectedOption: (selectedOption, length) => `${selectedOption} (1 of ${length}) is selected.`,
tSelectedOption: (selectedOption, length, index) => `${selectedOption} (${index + 1} of ${length}) is selected.`,
tResults: (length, contentSelectedOption) => {
const words = {
result: (length === 1) ? 'result' : 'results',
Expand Down Expand Up @@ -32,6 +32,7 @@ export default class Status extends Component {
queryLength,
minQueryLength,
selectedOption,
selectedOptionIndex,
tQueryTooShort,
tNoResults,
tSelectedOption,
Expand All @@ -43,7 +44,7 @@ export default class Status extends Component {
const noResults = length === 0

const contentSelectedOption = selectedOption
? tSelectedOption(selectedOption, length)
? tSelectedOption(selectedOption, length, selectedOptionIndex)
: ''

let content = null
Expand Down

0 comments on commit 2ae9940

Please sign in to comment.