Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2023-02-25 Thread Artem Semenov
On Wed, 9 Nov 2022 09:42:00 GMT, Artem Semenov wrote: >> A11Y implementation on macOS has to directly call the >> 'JList.setSelectedIndex' method in order to request selection on an item >> (see 'CAccessibility.requestSelection'). The reason is that a11y API lacks >> appropriate method.There's

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2023-02-22 Thread Artem Semenov
On Wed, 9 Nov 2022 09:42:00 GMT, Artem Semenov wrote: >> A11Y implementation on macOS has to directly call the >> 'JList.setSelectedIndex' method in order to request selection on an item >> (see 'CAccessibility.requestSelection'). The reason is that a11y API lacks >> appropriate method.There's

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2023-01-24 Thread Artem Semenov
On Wed, 9 Nov 2022 09:42:00 GMT, Artem Semenov wrote: >> A11Y implementation on macOS has to directly call the >> 'JList.setSelectedIndex' method in order to request selection on an item >> (see 'CAccessibility.requestSelection'). The reason is that a11y API lacks >> appropriate method.There's

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-12-26 Thread Artem Semenov
On Wed, 9 Nov 2022 09:42:00 GMT, Artem Semenov wrote: >> A11Y implementation on macOS has to directly call the >> 'JList.setSelectedIndex' method in order to request selection on an item >> (see 'CAccessibility.requestSelection'). The reason is that a11y API lacks >> appropriate method.There's

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-28 Thread Artem Semenov
On Wed, 9 Nov 2022 09:42:00 GMT, Artem Semenov wrote: >> A11Y implementation on macOS has to directly call the >> 'JList.setSelectedIndex' method in order to request selection on an item >> (see 'CAccessibility.requestSelection'). The reason is that a11y API lacks >> appropriate method.There's

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-22 Thread Sergey Bylokhov
On Wed, 23 Nov 2022 02:16:31 GMT, Sergey Bylokhov wrote: >> I tried calling clearSelection again instead of adding an interface: >> >> if (as == null) return; >> if (pac.getAccessibleRole().equals(AccessibleRole.LIST)) { >> as.clearAccessibleSe

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-22 Thread Sergey Bylokhov
On Mon, 21 Nov 2022 10:44:43 GMT, Artem Semenov wrote: >> It is unclear why the `setSelectionInterval `is actually any better than >> `addAccessibleSelection`. Both have a similar implementation, in some >> cases, the addSelectionInterval just calls setSelectionInterval, and seems >> that pos

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-21 Thread Artem Semenov
On Sun, 20 Nov 2022 03:16:28 GMT, Sergey Bylokhov wrote: >> It looks like it is. Please formulate the question differently. > > It is unclear why the `setSelectionInterval `is actually any better than > `addAccessibleSelection`. Both have a similar implementation, in some cases, > the addSelec

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-19 Thread Sergey Bylokhov
On Mon, 14 Nov 2022 11:07:56 GMT, Artem Semenov wrote: >> Probably because of the endless notifications loop? implementation of >> addSelectionInterval looks similar to setSelectionInterval. In some cases >> the addSelectionInterval just call setSelectionInterval. > > It looks like it is. Pleas

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-14 Thread Artem Semenov
On Fri, 11 Nov 2022 21:40:40 GMT, Sergey Bylokhov wrote: >> We tried to do this, it leads to an endless repetition of the selected line >> until it freezes. > > Probably because of the endless notifications loop? implementation of > addSelectionInterval looks similar to setSelectionInterval. In

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-11 Thread Sergey Bylokhov
On Fri, 11 Nov 2022 09:12:21 GMT, Artem Semenov wrote: >> src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessibility.java line >> 574: >> >>> 572: return; >>> 573: } >>> 574: as.addAccessibleSelection(i); >> >> I would like to clarify on

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-11 Thread Artem Semenov
On Fri, 11 Nov 2022 02:16:15 GMT, Sergey Bylokhov wrote: >> Artem Semenov has updated the pull request incrementally with one additional >> commit since the last revision: >> >> We are not using author tags in tests either - but that's just a nitpick. > > src/java.desktop/macosx/classes/sun/l

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-10 Thread Sergey Bylokhov
On Wed, 9 Nov 2022 09:42:00 GMT, Artem Semenov wrote: >> A11Y implementation on macOS has to directly call the >> 'JList.setSelectedIndex' method in order to request selection on an item >> (see 'CAccessibility.requestSelection'). The reason is that a11y API lacks >> appropriate method.There's

Re: RFR: 8271846 a11y API lacks setSelectedIndex method [v4]

2022-11-09 Thread Artem Semenov
> A11Y implementation on macOS has to directly call the > 'JList.setSelectedIndex' method in order to request selection on an item (see > 'CAccessibility.requestSelection'). The reason is that a11y API lacks > appropriate method.There's only > 'javax.accessibility.AccessibleSelection#addAccessi