-
Notifications
You must be signed in to change notification settings - Fork 20
TypeScript and class Combobox #22
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
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.
👍 This will be a breaking change, but so was #21 so we can release as one set of breaking changes.
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.
This looks great to me ✨
@jscholes Hello! here's a new demo page for this pull. The list should expand if the input isn't empty. |
@muan This is looking good! A few things which don't seem to work:
|
I can add this logic to the demo page, but to be clear this won't be an out-of-box function for
I'll give this a go.
I'll add clearing selection on non-captured keydown events.
I'll consider these when making this change to
I'll try doing the first. Though I noticed that the spec matches your suggestion, but oddly all the APG examples wraps. |
These interactions are not used to interact with list results, and we assume they are use to interact with the input, therefore we unselect items.
options are not li elements
Thanks for the feedback @jscholes!
I've added this to the demo page. Though when testing in Safari with Voice Over, the options aren't read out if the input is empty.
I've added these, and the demo page is updated. Finally..
I tried out implementing this, but we don't really know how list close should work (ie is it removed? is it clear? is it hidden? how was it hid?). To avoid assuming how, we'd need the components to pass in a function for how they want it handled, but inevitably there will still be behaviors that are handled by the component that require the same logic, which are then evaluated in two places.
I hear you, but as it stands now, all our components using For example, On the demo page, I've added close on Enter and added aria-live to ensure the commit event logger is announced. |
@muan Test page is looking good!
This is odd. Unless I'm mistaken, all of the use cases you've outlined for GitHub require text to be typed before the combo box is useful. Is that correct, including for ones not mentioned as examples? If so, this doesn't matter right now, but this should probably be investigated and reported. I wonder if the value of
Makes sense. |
Yes.
I'll create an isolated test page for this and see if I can track this down. |
This pull request includes the changes for
Before:
After: