Bug 97127 – AX: combo box should be writable on Mac

• Status: NEW
• Component: Accessibility

[Dominic Mazzoni]

From bug 96735:
> combo boxes should be writeable and not read only (because you can type into the combo box)
>
> it looks like list boxes AXSelectedChildren should be writeable (but not its AXValue)

Chris, can you capture any other attributes whose writable state seems to be wrong now on Mac? Maybe we can clean up several at once.

I wonder if we could split the cross-platform interface into:

* isValueModifiable - independent of whether it's contentEditable or enabled, should this role have a modifiable value in general?
* isContentEditable
* isEnabled
* isEditableTextField

I think each platform could compute its correct values for "read only" based on these four. I believe we wouldn't need any special casing in cross-platform code then.

[chris fleizach]

(In reply to comment #0)
> From bug 96735:
> > combo boxes should be writeable and not read only (because you can type into the combo box)
> >
> > it looks like list boxes AXSelectedChildren should be writeable (but not its AXValue)
> 
> Chris, can you capture any other attributes whose writable state seems to be wrong now on Mac? Maybe we can clean up several at once.
> 

As far as I can tell, combo box is the only other control that is not read only (but marked as such in webkit)

> I wonder if we could split the cross-platform interface into:
> 
> * isValueModifiable - independent of whether it's contentEditable or enabled, should this role have a modifiable value in general?
> * isContentEditable
> * isEnabled
> * isEditableTextField
> 

It does seem like read only can have different meanings based on different items or contexts.

isValueModifiable sounds like it would be the natural replacement for most isReadOnly calls

isEditableTextField and isContentEditable seems like they could be merged

Obviously isEnabled is there already, and used for another type of attribte on the Mac (AXEnabled)

> I think each platform could compute its correct values for "read only" based on these four. I believe we wouldn't need any special casing in cross-platform code then.

[Radar WebKit Bug Importer]

<rdar://problem/15710436>