New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix long clicks not selecting options #410
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a CHANGELOG entry
c75aaea
to
2ae3f55
Compare
2ae3f55
to
670f9ca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work getting to the bottom of this 👍
I've done some testing in IE11 (Windows 10), reproduced the original issue in the country and territory autocomplete, and tested the country and territory autocomplete with this branch installed.
Seems to fix the issue without reintroducing the original problem.
### Fixes - [Pull request #388: Set aria-selected as a string instead of a boolean to avoid being dropped.](#388) - [Pull request #400: Remove pointer events check.](#400) - [Pull request #406: Make hint padding match input padding.](#406) - [Pull request #407: Use a div element to wrap enhanced component.](#407) - [Pull request #410: Fix long clicks not selecting options.](#410)
### Fixes - [Pull request #388: Set aria-selected as a string instead of a boolean to avoid being dropped.](#388) - [Pull request #400: Remove pointer events check.](#400) - [Pull request #406: Make hint padding match input padding.](#406) - [Pull request #407: Use a div element to wrap enhanced component.](#407) - [Pull request #410: Fix long clicks not selecting options.](#410)
### Fixes - [Pull request alphagov#388: Set aria-selected as a string instead of a boolean to avoid being dropped.](alphagov#388) - [Pull request alphagov#400: Remove pointer events check.](alphagov#400) - [Pull request alphagov#406: Make hint padding match input padding.](alphagov#406) - [Pull request alphagov#407: Use a div element to wrap enhanced component.](alphagov#407) - [Pull request alphagov#410: Fix long clicks not selecting options.](alphagov#410)
### Fixes - [Pull request alphagov#388: Set aria-selected as a string instead of a boolean to avoid being dropped.](alphagov#388) - [Pull request alphagov#400: Remove pointer events check.](alphagov#400) - [Pull request alphagov#406: Make hint padding match input padding.](alphagov#406) - [Pull request alphagov#407: Use a div element to wrap enhanced component.](alphagov#407) - [Pull request alphagov#410: Fix long clicks not selecting options.](alphagov#410)
Through exploring this issue I've determined that a change to fix a bug is the reason why doing a long click will not work on the autocomplete since the click needs to finish in 200ms.
I've explored an alternative but tried reverting this specific commit and then testing custom templates in Internet Explorer 11 and cannot reproduce the original issue.
Fixes #408
Fixes #328
Closes #409 (alternative try at fixing this)
Closes #342 (alternative try at fixing this)