and analogous change to selectedChildren, as you pointed out, to read as 
follows:

selectedCelss([in] long maxCells,[out, size_is(, maxCells), 
length_is(,*nselCells)] IUnknown** ppCells,[out, retval] long *nselCells)
Best regards,

--Andres.



________________________________
From: [email protected] 
[mailto:[email protected]] On Behalf Of 
Andres Gonzalez
Sent: Monday, July 06, 2009 3:49 PM
To: [email protected]; IA2 List
Subject: Re: [Accessibility-ia2] IATable changes


Pete and all: just to clarify, my proposal is to change the following two 
method signatures to read as follows:



HRESULT columnIndex([in] IUnknown* pcell,[out, retval] long *columnIndex);

HRESULT rowIndex([in] IUnknown* pcell,[out, retval] long *rowIndex);



which will return an error value (e.g., E_INVALIDARG, or something like that) 
if pcell doesn't point to an IAccessible or IAccessible2, or if the object it 
points to is not part of the table. I'm totally fine with changing IUnknown to 
IAccessible or IAccessible2 above.



With that change, we can then get rid of the following method altogether, 
because we don't need it anymore:



HRESULT childIndex([in] long rowIndex,[in] long columnIndex,[out, retval] long 
*cellIndex);


We don't need this concept of cellIndex no more, which in my opinion, was an 
unnecessary design constraint of the original interface.

Best regards,

--Andres.



________________________________
From: [email protected] 
[mailto:[email protected]] On Behalf Of Pete 
Brunet
Sent: Monday, July 06, 2009 8:09 AM
To: IA2 List
Subject: [Accessibility-ia2] IATable changes

Since we are discussing an update to the table interface to provide a new 
method to get a list of header cells for specified cell index and since that 
will cause the creation of IAccessibleTable2 we need to discuss related IATable 
issues.

1) Since cells are not necessarily children, rename the following methods:

childIndex to cellIndex
selectedChildren to selectedCells
nSelectedChildren to nSelectedCells

Note: We've already changed the parameter names and related docs to use the 
word cell instead of child but didn't want to change the method names up to 
this point so as to not have to rev the IDL.

2) During our meeting of April 29, 2008 Andres said he would like to see the 
use of cell indexes replaced with references to a cell's accessible. For 
example, instead of giving the table a cell index and asking for the row and 
column indexes, give the table a reference to the cell and and let the table 
return the row and column indexes.

This would impact the following methods:

childIndex ([in] long rowIndex,[in] long columnIndex,[out, retval] long 
*cellIndex)
    return a reference to an accessible (actually an IUknown)
columnIndex ([in] long cellIndex,[out, retval] long *columnIndex)
    change the [in] to a reference to an accessible (an IUknown)
rowIndex ([in] long cellIndex,[out, retval] long *rowIndex)
    ditto
selectedChildren ([in] long maxChildren,[out, size_is(, maxChildren), 
length_is(,*nChildren)] long **children,[out, retval] long *nChildren)
    return an array of references to IUnknowns

What are your thoughts on these changes?
--
Pete Brunet

a11ysoft - Accessibility Architecture and Development
(512) 238-6967
pete @ a11ysoft.com
http://www.a11ysoft.com/about/
http://www.linkedin.com/in/petebrunet
Ionosphere: WS4G
_______________________________________________
Accessibility-ia2 mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/accessibility-ia2

Reply via email to