Closed dmcnamara-eng closed 2 years ago
@NullVoxPopuli Heyo, I'm looking to get this PR up-to-date with latest master and eventually merge it, adding Combobox
to the list of supported add-ons. Could you approve and run this PR for me, please? Thank you.
@balinterdi yea, I'll keep the tab open -- looks like ci already ran tho!
Closed in favour of https://github.com/GavinJoyce/ember-headlessui/pull/165
Β β¨ What Changed and Why
This PR is a functionally complete implementation of a
<Combobox>
component based on https://headlessui.dev/react/comboboxThings that this PR implements:
Enter
/Space
/ArrowUp
/ArrowDown
/ArrowLeft
/ArrowRight
/PageUp
/PageDown
/Home
/End
/Escape
)aria-expanded
/aria-labelledby
/aria-controls
/aria-haspopup
/aria-active-descendent
) and focus managementI ported the React component tests for combobox as Ember integration tests. 122 integration tests were ported. Some tests are marked as
skip
ortodo
because they are react specific or covered elesewhere.π Before
Didn't exist.
π After
https://user-images.githubusercontent.com/70960245/163006210-6fccc1ed-af45-41d6-b04c-ebd453eeeeea.mov
π Testing
Closes #138