Joanmarie Diggs Assignee
	Description • 7 years ago

In ARIA 1.1, aria-orientation was added to the following roles with the specified implicit values:
* combobox: undefined
* listbox: vertical
* menu: vertical
* menubar: horizontal
* radiogroup: undefined
* tablist: horizontal
* toolbar: horizontal
* tree: vertical
* treegrid: undefined

In ARIA 1.1, the following changes related to aria-orientation were also made:
* aria-orientation's default value changed from horizontal to undefined
* separator now has implicit value of horizontal
* slider now has implicit value of horizontal

Please note, the following steps to reproduce only test a couple of roles for their implicit values. At the time of this report, automated tests to verify correct exposure are still being written.

Steps to reproduce:
1. Load data:text/html,<div role="treegrid" />
2. Use Accerciser to examine the state set of the accessible element
3. Load data:text/html,<div role="combobox" />
4. Use Accerciser to examine the state set of the accessible element

Expected result: The state set would not contain STATE_HORIZONTAL or STATE_VERTICAL for either test case, because neither orientation value is specified and the implicit value of aria-orientation for treegrid and combobox are "undefined."

Actual result: The state set contains STATE_VERTICAL for the two test cases.

	alexander :surkov (:asurkov)
	Comment 2 • 7 years ago

Comment on attachment 8879524 [details] [diff] [review]
proposed patch

Review of attachment 8879524 [details] [diff] [review]:
-----------------------------------------------------------------

r=me, thanks!

	Joanmarie Diggs Assignee
	Comment 3 • 7 years ago

Comment on attachment 8879524 [details] [diff] [review]
proposed patch

Clearing the checkin flag. I thought that was "checkin needed". Sorry!

	Pulsebot
	Comment 4 • 7 years ago

Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/343e7dbd5be2
Implement or update support for exposure of aria-orientation. r=surkov

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 5 • 7 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/343e7dbd5be2