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
