-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
selectlist: implement new keyboard behavior
Fixed: 1422275 Change-Id: If5e7328ad739f9c7339dcd17561c57875d4255e7
- Loading branch information
1 parent
ec15f83
commit ab8d967
Showing
1 changed file
with
163 additions
and
0 deletions.
There are no files selected for viewing
163 changes: 163 additions & 0 deletions
163
html/semantics/forms/the-selectlist-element/selectlist-keyboard-behavior.tentative.html
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,163 @@ | ||
| <!DOCTYPE html> | ||
| <link rel=author href="mailto:jarhar@chromium.org"> | ||
| <link rel=help href="https://bugs.chromium.org/p/chromium/issues/detail?id=1422275"> | ||
| <link rel=help href="https://github.com/openui/open-ui/issues/433#issuecomment-1452461404"> | ||
| <link rel=help href="https://github.com/openui/open-ui/issues/386#issuecomment-1452469497"> | ||
| <script src="/resources/testharness.js"></script> | ||
| <script src="/resources/testharnessreport.js"></script> | ||
| <script src="/resources/testdriver.js"></script> | ||
| <script src="/resources/testdriver-vendor.js"></script> | ||
|
|
||
| <form> | ||
| </form> | ||
|
|
||
| <div id=notform> | ||
| <selectlist id=defaultbutton> | ||
| <option class=one>one</option> | ||
| <option class=two>two</option> | ||
| <option class=three>three</option> | ||
| </selectlist> | ||
|
|
||
| <selectlist id=custombutton> | ||
| <button type=selectlist>custom button</button> | ||
| <option class=one>one</option> | ||
| <option class=two>two</option> | ||
| <option class=three>three</option> | ||
| </selectlist> | ||
| </div> | ||
|
|
||
| <script> | ||
| const Enter = '\uE007'; | ||
| const Escape = '\uE00C'; | ||
| const ArrowLeft = '\uE012'; | ||
| const ArrowUp = '\uE013'; | ||
| const ArrowRight = '\uE014'; | ||
| const ArrowDown = '\uE015'; | ||
| const Space = ' '; | ||
| const form = document.querySelector('form'); | ||
| const notform = document.getElementById('notform'); | ||
|
|
||
| for (const id of ['defaultbutton', 'custombutton']) { | ||
| const selectlist = document.getElementById(id); | ||
|
|
||
| async function closeListbox() { | ||
| await test_driver.click(selectlist); | ||
| } | ||
|
|
||
| function addCloseCleanup(t) { | ||
| t.add_cleanup(async () => { | ||
| if (selectlist.matches(':open')) { | ||
| await closeListbox(); | ||
| } | ||
| if (selectlist.matches(':open')) { | ||
| throw new Error('selectlist failed to close!'); | ||
| } | ||
| selectlist.value = 'one'; | ||
| }); | ||
| } | ||
|
|
||
| promise_test(async t => { | ||
| addCloseCleanup(t); | ||
| selectlist.focus(); | ||
| assert_false(selectlist.matches(':open'), | ||
| 'The selectlist should initially be closed.'); | ||
| await test_driver.send_keys(selectlist, Space); | ||
| assert_true(selectlist.matches(':open'), | ||
| 'The selectlist should be open after pressing space.'); | ||
| }, `${id}: When the listbox is closed, spacebar should open the listbox.`); | ||
|
|
||
| promise_test(async t => { | ||
| addCloseCleanup(t); | ||
| selectlist.value = 'two'; | ||
| selectlist.focus(); | ||
| assert_false(selectlist.matches(':open'), | ||
| 'The selectlist should initially be closed.'); | ||
|
|
||
| await test_driver.send_keys(selectlist, ArrowLeft); | ||
| assert_true(selectlist.matches(':open'), | ||
| 'Arrow left should open the listbox.'); | ||
| assert_equals(selectlist.value, 'two', | ||
| 'Arrow left should not change the selected value.'); | ||
| await closeListbox(); | ||
|
|
||
| await test_driver.send_keys(selectlist, ArrowUp); | ||
| assert_true(selectlist.matches(':open'), | ||
| 'Arrow up should open the listbox.'); | ||
| assert_equals(selectlist.value, 'two', | ||
| 'Arrow up should not change the selected value.'); | ||
| await closeListbox(); | ||
|
|
||
| await test_driver.send_keys(selectlist, ArrowRight); | ||
| assert_true(selectlist.matches(':open'), | ||
| 'Arrow right should open the listbox.'); | ||
| assert_equals(selectlist.value, 'two', | ||
| 'Arrow right should not change the selected value.'); | ||
| await closeListbox(); | ||
|
|
||
| await test_driver.send_keys(selectlist, ArrowDown); | ||
| assert_true(selectlist.matches(':open'), | ||
| 'Arrow down should open the listbox.'); | ||
| assert_equals(selectlist.value, 'two', | ||
| 'Arrow down should not change the selected value.'); | ||
| }, `${id}: When the listbox is closed, all arrow keys should open the listbox.`); | ||
|
|
||
| promise_test(async t => { | ||
| addCloseCleanup(t); | ||
|
|
||
| selectlist.focus(); | ||
| await test_driver.send_keys(selectlist, Enter); | ||
| assert_false(selectlist.matches(':open'), | ||
| 'Enter should not open the listbox when outside a form.'); | ||
|
|
||
| form.appendChild(selectlist); | ||
| let formWasSubmitted = false; | ||
| form.addEventListener('form', event => { | ||
| event.preventDefault(); | ||
| formWasSubmitted = true; | ||
| }, {once: true}); | ||
| await test_driver.send_keys(selectlist, Enter); | ||
| assert_true(formWasSubmitted, | ||
| 'Enter should submit the form when the listbox is closed.'); | ||
| assert_false(selectlist.matches(':open'), | ||
| 'Enter should not open the listbox when it is in a form.'); | ||
| }, `${id}: When the listbox is closed, the enter key should submit the form or do nothing.`); | ||
|
|
||
| promise_test(async t => { | ||
| addCloseCleanup(t); | ||
| const optionOne = selectlist.querySelector('.one'); | ||
| const optionTwo = selectlist.querySelector('.two'); | ||
| const optionThree = seelctlist.querySelector('.three'); | ||
|
|
||
| selectlist.value = 'two'; | ||
| await test_driver.click(selectlist); | ||
| assert_true(selectlist.matches(':open'), | ||
| 'The selectlist should open when clicked.'); | ||
| asset_equals(document.activeElement, optionTwo, | ||
| 'The selected option should receive initial focus.'); | ||
|
|
||
| await test_driver.send_keys(document.activeElement, ArrowDown); | ||
| assert_equals(document.activeElement, optionThree, | ||
| 'The next option should receive focus when the down arrow key is pressed.'); | ||
| assert_equals(selectlist.value, 'two', | ||
| 'The selectlists value should not change when focusing another option.'); | ||
|
|
||
| await test_driver.send_keys(document.activeElement, ArrowUp); | ||
| assert_equals(document.activeElement, optionTwo, | ||
| 'The previous option should receive focus when the up arrow key is pressed.'); | ||
| assert_equals(selectlist.value, 'two', | ||
| 'The selectlists value should not change when focusing another option.'); | ||
|
|
||
| await test_driver.send_keys(document.activeElement, ArrowUp); | ||
| assert_equals(document.activeElement, optionOne, | ||
| 'The first option should be selected.'); | ||
| assert_equals(selectlist.value, 'two', | ||
| 'The selectlists value should not change when focusing another option.'); | ||
|
|
||
| await test_driver.send_keys(document.activeElement, Enter); | ||
| assert_false(selectlist.matches(':open'), | ||
| 'The listbox should be closed after pressing enter.'); | ||
| assert_equals(selectlist.value, 'one', | ||
| 'The selectlists value should change after pressing enter on a different option.'); | ||
| }, `{id}: When the listbox is open, the enter key should commit the selected option.`); | ||
| } | ||
| </script> |