This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 19772 - if statement not needed for cobobox MSAA+UIA Express
Summary: if statement not needed for cobobox MSAA+UIA Express
Status: CLOSED FIXED
Alias: None
Product: ARIA
Classification: Unclassified
Component: Core AAM (show other bugs)
Version: 1.0
Hardware: PC Windows NT
: P2 normal
Target Milestone: ---
Assignee: Cynthia Shelly
QA Contact: ARIA UA Implementors
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-30 10:16 UTC by Cynthia Shelly
Modified: 2013-06-13 17:19 UTC (History)
4 users (show)

See Also:


Attachments

Description Cynthia Shelly 2012-10-30 10:16:02 UTC
remove "If aria-expanded (state) is not "true", expose STATE_SYSTEM_COLLAPSED' from UIA Express column for combox.  This is normal behavior for aria-expanded property and is covered in property table.

Check with David if this is needed for IA2.
Comment 1 Andi Snow-Weaver 2012-11-06 17:33:19 UTC
Done for MSAA+UIAExpress.

David, does this text need to be removed from the MSAA+IA2 column?
Comment 2 alexander surkov 2012-11-08 01:27:26 UTC
It's not really covered by aria-expanded mapping:

"aria-expanded is undefined (state) (default)" has "Not mapped" for all platforms including both UIA and IA2.

Bug doesn't seems valid.
Comment 3 Andi Snow-Weaver 2012-11-08 14:55:59 UTC
Cynthia,

Can you please respond to Alex's comment in the bug? Do we need to change the aria-expanded undefined row now that we have removed this text?
Comment 4 David Bolter 2013-02-21 18:23:32 UTC
(In reply to comment #2)
> It's not really covered by aria-expanded mapping:
> 
> "aria-expanded is undefined (state) (default)" has "Not mapped" for all
> platforms including both UIA and IA2.

Alex we discussed this briefly during a W3C call but weren't 100% sure what you meant here.

> 
> Bug doesn't seems valid.


Sounds like we can close this bug as INVALID/WORKSFORME right? (I'd agree)
Comment 5 alexander surkov 2013-02-22 02:05:59 UTC
(In reply to comment #4)
> (In reply to comment #2)
> > It's not really covered by aria-expanded mapping:
> > 
> > "aria-expanded is undefined (state) (default)" has "Not mapped" for all
> > platforms including both UIA and IA2.
> 
> Alex we discussed this briefly during a W3C call but weren't 100% sure what
> you meant here.

I don't recall details. But I think the problem is.

Cynthia asked to remove "If aria-expanded (state) is not "true", expose STATE_SYSTEM_COLLAPSED' from UIA Express column for combox" because "This is normal behavior for aria-expanded property and is covered in property table". However aria-expanded property table says "aria-expanded is undefined (state) (default)" has "Not mapped" for all platforms". That means if those words are removed then combobox having not "true" aria-expanded shouldn't expose STATE_SYSTEM_COLLAPSED.

> > Bug doesn't seems valid.
> 
> Sounds like we can close this bug as INVALID/WORKSFORME right? (I'd agree)

You can't call it invalid or worksforme until you back out the fix in MSAA+UIAExpress case (see comment #1).
Comment 6 David Bolter 2013-03-21 18:22:04 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > (In reply to comment #2)
> > > It's not really covered by aria-expanded mapping:
> > > 
> > > "aria-expanded is undefined (state) (default)" has "Not mapped" for all
> > > platforms including both UIA and IA2.
> > 
> > Alex we discussed this briefly during a W3C call but weren't 100% sure what
> > you meant here.
> 
> I don't recall details. But I think the problem is.
> 
> Cynthia asked to remove "If aria-expanded (state) is not "true", expose
> STATE_SYSTEM_COLLAPSED' from UIA Express column for combox" because "This is
> normal behavior for aria-expanded property and is covered in property
> table". However aria-expanded property table says "aria-expanded is
> undefined (state) (default)" has "Not mapped" for all platforms". That means
> if those words are removed then combobox having not "true" aria-expanded
> shouldn't expose STATE_SYSTEM_COLLAPSED.

This made my head hurt but I think I got it :)

Cynthia are you sure you don't want to state that UIA Express should expose STATE_SYSTEM_COLLAPSED for the case where role=combobox and aria-expanded is undefined?

> 
> > > Bug doesn't seems valid.
> > 
> > Sounds like we can close this bug as INVALID/WORKSFORME right? (I'd agree)
> 
> You can't call it invalid or worksforme until you back out the fix in
> MSAA+UIAExpress case (see comment #1).

It took me a while but I think here you are simply pointing out a technicality that if we close this as is then it would be as "FIXED" not INVALID/WORKSFORME right?

Alex are you OK leaving this one up to Cynthia/MS or are you concerned about this text WRT a possible FF UI Express implementation?
Comment 7 alexander surkov 2013-03-22 04:25:58 UTC
If we leave out it as it is then it means that

<div role="combobox"> is neither collapsed nor expanded in UIA Express and it is collapsed in IA2. In other words IA2 and UIA Express have different requirements about MSAA states. In case of Firefox both IA2 and UIA Express shares same MSAA tree so that will make us to break the spec this or that way.

On the another hand if UIA bridge doesn't guess that neither collapsed nor expanded combobox is actually collapsed then screen readers should have a different logic for combobox processing depending on API. I don't think more diversity in this case is sensible.

Anyway we could do a guessing more but it'd be great to hear Cynthia. It seems there is a bunch of options here:
1) It was done on purpose to comply with some UIA requirements
2) I missed some part of the spec and my guesses are wrong
3) It was a mistake
Comment 8 Joseph Scheuhammer 2013-03-22 16:52:12 UTC
Here's another consideration.  The spec for combobox asserts that aria-expanded is a *required* state [1] -- see the "Characteristics of combobox table".  The implication is that comboboxes are inherently expandable/collapsible.

That, in turn, means a combobox is either expanded or collapsed.  I think it's a safe bet that when first instantiated, the combobox is collapsed.  Rather, I can't think of a use case where a combobox is initially expanded without the user interacting with it. (It's possible that I'm just not creative enough...).

This logic leads back to "if aria-expanded is not 'true', expose STATE_SYSTEM_COLLAPSED".

[1] http://www.w3.org/WAI/PF/aria/roles#combobox
Comment 9 alexander surkov 2013-03-23 01:47:21 UTC
Shouldn't UAIG explicitly say that collapsed combobox means SYSTEM_STATE_COLLAPSED must be exposed in case of MSAA+UIA/MSAA+IA2 ?
Comment 10 Joseph Scheuhammer 2013-03-25 15:19:08 UTC
(In reply to comment #9)
> Shouldn't UAIG explicitly say that collapsed combobox means
> SYSTEM_STATE_COLLAPSED must be exposed in case of MSAA+UIA/MSAA+IA2 ?

It does for MSAA+IA2.
It doesn't for MSAA+UIA because Cynthia requested it be removed (the reason for this bugzilla).

IMHO, that edit should be undone; but, waiting for Cynthia's opinion.
Comment 11 Cynthia Shelly 2013-03-28 17:32:50 UTC
Go ahead and put it back, pending david's approval of Issue 566.
Comment 12 David Bolter 2013-05-02 17:31:58 UTC
(In reply to comment #11)
> Go ahead and put it back, pending david's approval of Issue 566.

Sure I'm fine with reintroducing text like: 'If aria-expanded (state) is not "true", expose STATE_SYSTEM_COLLAPSED' to be put in the UIA Express Column for role combobox. (Issue 566)
Comment 13 Joseph Scheuhammer 2013-05-02 17:32:54 UTC
(In reply to comment #12)
> (In reply to comment #11)
> > Go ahead and put it back, pending david's approval of Issue 566.
> 
> Sure I'm fine with reintroducing text like: 'If aria-expanded (state) is not
> "true", expose STATE_SYSTEM_COLLAPSED' to be put in the UIA Express Column
> for role combobox. (Issue 566)

Cynthia, feel free to close this given David's comment.