Skip to content
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

Hide the HoneypotField from assistive technology (AT) #26

Merged
merged 1 commit into from
Jun 21, 2019

Conversation

polarbirke
Copy link
Contributor

Users of AT (e.g. screen readers) navigate with arrow keys which do not
(always) respect tabindex=-1, so the additional ARIA attribute is necessary.

Users of AT (e.g. screen readers) navigate with arrow keys which do not
(always) respect tabindex=-1, so the additional ARIA attribute is necessary.
@eymengunay eymengunay merged commit e4e2ae9 into eymengunay:master Jun 21, 2019
@eymengunay
Copy link
Owner

eymengunay commented Jun 21, 2019

Thank you @polarbirke, it is merged!

https://github.com/eymengunay/EoHoneypotBundle/releases/tag/v1.2.3

@polarbirke
Copy link
Contributor Author

Hi @eymengunay, can you trigger an update for https://packagist.org/packages/eo/honeypot-bundle as well? Apparently the release wasn't published yet. Thank you!

@eymengunay
Copy link
Owner

Hey @polarbirke , Sorry for the late response.

I don't know why it didn't get refreshed automatically but I see that it is now updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants