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

Escape should clear text in input #680

Closed
silviuaavram opened this issue Feb 21, 2019 · 10 comments · Fixed by #689
Closed

Escape should clear text in input #680

silviuaavram opened this issue Feb 21, 2019 · 10 comments · Fixed by #689

Comments

@silviuaavram
Copy link
Collaborator

https://www.w3.org/TR/wai-aria-practices/examples/combobox/aria1.1pattern/listbox-combo.html

Escape key inside input should clear the text from it. Currently we are resetting it to the string value of the item.

@babzcraig
Copy link

If no one is on this yet, I'd like to take this on @silviuavram. I've read the requirements on the w3 page and can make a PR for this

@silviuaavram
Copy link
Collaborator Author

That is great to hear! It's all yours, let me know when I can help out. Thanks!

@silviuaavram
Copy link
Collaborator Author

@babzcraig how is the progress on this one? need any help? :)

@gaskar
Copy link
Contributor

gaskar commented Mar 15, 2019

hi @silviuavram I can take this one.

@babzcraig
Copy link

@gaskar please feel free to. I couldn't get downshift's validate-setup command to work and I've been too swamped to debug for a while. @silviuavram perhaps you can help me with this and I can take on another issue. Thanks!

gaskar added a commit to gaskar/downshift that referenced this issue Mar 15, 2019
@gaskar
Copy link
Contributor

gaskar commented Mar 15, 2019

@babzcraig seems there is a problem with cypress tests, I've tried different versions nothing is working beside of just commenting out these tests (or using workaround with catching uncaught exceptions), seems that @silviuavram created PR for this problem but it is failing as well
#661

@babzcraig
Copy link

I see. I had just changed my development machine so I though maybe I hadn't set up things well yet. I might have to do the same then. If you want to take this issue, please do. I won't be available until Sunday so I'll just pick another issue then if you're on this

gaskar added a commit to gaskar/downshift that referenced this issue Apr 21, 2019
@silviuaavram
Copy link
Collaborator Author

🎉 This issue has been resolved in version 3.2.10 🎉

The release is available on:

Your semantic-release bot 📦🚀

@brandonkal
Copy link

This should be reconsidered. Escape is usually used to blur an element. Clearing the textbox is an unexpected side effect.

See: w3c/aria-practices#1066
Reverting this is already planned for aria-practices 1.2

Just to provide some additional examples, try pressing escape in the "Search or jump to..." box above or any search engine.

The only other place I have seen this pattern is in the Mac System Preferences search box, but this is done in tandem with removing focus on the input, which is the expected behavior.

@filipjnc
Copy link

filipjnc commented Sep 5, 2019

Please reopen the issue, as there is a new official guideline by ARIA:
The Escape key should only clear the input, if the dropdown is not currently open.
Source: w3c/aria-practices#1129

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

Successfully merging a pull request may close this issue.

5 participants