Docs for autoselect are incorrect

[edwardhorsford]

The docs state that autoselect defaults to false - this does not appear to be the case at least for the accessibleAutocomplete.enhanceSelectElement() method is used - the feature is on by default.

This is problematic as autoselect is basically incompatible with show all values and causes usability issues.

[matthewford]

@edwardhorsford im writing a blog post on this atm with a fix in our branch

[matthewford]

Our branch master...bitzesty:select-behaviour

And a blog post on some of the changes we made. Once we rebase and update the docs we will raise a PR.

https://twitter.com/bitzesty/status/1309503796813484032

[36degrees]

It looks like the default value for autoselect is false:

accessible-autocomplete/src/autocomplete.js

Line 39 in d617d5f

autoselect: false,

However, enhanceSelectElement specifically enables it (unless it's passed in as an option):

accessible-autocomplete/src/wrapper.js

Line 45 in d617d5f

if (configurationOptions.autoselect === undefined) configurationOptions.autoselect = true

It currently says that it will do this in the README as well:

You can use the accessibleAutocomplete.enhanceSelectElement function to enhance it into an autocomplete:

accessibleAutocomplete.enhanceSelectElement({
  selectElement: document.querySelector('#location-picker')
})

This will:

• Place an autocomplete input field after the specified <select>
• Default the autocomplete autoselect to true
• [...]

So I believe the docs are currently correct, even if the behaviour isn't desired.